Address review comments

Signed-off-by: Sara Adams <sara.e.adams@gmail.com>
EngFlow · Nov 27, 2023 · ca2fb3f · ca2fb3f
1 parent bf6d2a2
commit ca2fb3f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/analyzer/java/com/engflow/bazel/invocation/analyzer/bazelprofile/BazelEventsUtil.java b/analyzer/java/com/engflow/bazel/invocation/analyzer/bazelprofile/BazelEventsUtil.java
@@ -26,7 +26,7 @@
 
 import com.engflow.bazel.invocation.analyzer.traceeventformat.CompleteEvent;
 
-public class BazelEventsUtil {
+public final class BazelEventsUtil {
   private BazelEventsUtil() {}
 
   /** The event indicates that an action was executed locally. */
@@ -43,7 +43,7 @@ public static boolean indicatesRemoteExecution(CompleteEvent event) {
 
   /**
    * The event documents a remote cache check. This includes cache checks against remote caches
-   * configured by `--remote_cache` or `--disk_cache`.
+   * configured by {@code --remote_cache} or {@code --disk_cache}.
    */
   public static boolean indicatesRemoteCacheCheck(CompleteEvent event) {
     return CAT_REMOTE_ACTION_CACHE_CHECK.equals(event.category);

diff --git a/analyzer/java/com/engflow/bazel/invocation/analyzer/dataproviders/RemoteCacheMetrics.java b/analyzer/java/com/engflow/bazel/invocation/analyzer/dataproviders/RemoteCacheMetrics.java
@@ -25,8 +25,8 @@
  * Metrics on remote caching, namely how much time was spent on cache checks, downloading outputs,
  * uploading outputs, and what percentage of actions were cached remotely.
  *
- * <p>Note that Bazel does not differentiate between a remote cache configured via `--disk_cache` or
- * `--remote_cache`.
+ * <p>Note that Bazel does not differentiate between a remote cache configured via {@code
+ * --disk_cache} or {@code --remote_cache}.
  */
 public class RemoteCacheMetrics implements Datum {