commit 91ff7ec22d237836c53091a2a2a4a0bfb1b3a930
author Googler <noreply@google.com> Mon Dec 20 14:40:13 2021 -0800
committer Copybara-Service <copybara-worker@google.com> Mon Dec 20 14:41:55 2021 -0800
tree 03bdca4b6629346104e571a134df70cf8c59b706
parent 3cbfb39b3681266834f4057ab63005c736533205

Delete the option for using action keys and the resulting dead code

PiperOrigin-RevId: 417481923

src/main/java/com/google/devtools/build/lib/util/BigIntegerFingerprintUtils.java

diff --git a/src/main/java/com/google/devtools/build/lib/util/BigIntegerFingerprintUtils.java b/src/main/java/com/google/devtools/build/lib/util/BigIntegerFingerprintUtils.java
index 483519a..3fffe02 100644
--- a/src/main/java/com/google/devtools/build/lib/util/BigIntegerFingerprintUtils.java
+++ b/src/main/java/com/google/devtools/build/lib/util/BigIntegerFingerprintUtils.java
@@ -46,10 +46,6 @@
     return compose(v1, v2);
   }
 
-  public static BigInteger composeOrdered(BigInteger accumulator, BigInteger v) {
-    return compose(accumulator.multiply(RELATIVE_PRIME).mod(UINT128_LIMIT), v);
-  }
-
   @Nullable
   public static BigInteger composeOrderedNullable(
       @Nullable BigInteger accumulator, @Nullable BigInteger v) {

src/test/java/com/google/devtools/build/lib/actionsketch/ActionSketchTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/actionsketch/ActionSketchTest.java b/src/test/java/com/google/devtools/build/lib/actionsketch/ActionSketchTest.java
index 33bc4ed..ab3ea36 100644
--- a/src/test/java/com/google/devtools/build/lib/actionsketch/ActionSketchTest.java
+++ b/src/test/java/com/google/devtools/build/lib/actionsketch/ActionSketchTest.java
@@ -30,7 +30,6 @@
     roundTrip(
         ActionSketch.builder()
             .setTransitiveSourceHash(HashAndVersion.create(BigInteger.ONE, /*version=*/ 0L))
-            .setTransitiveActionLookupHash(new BigInteger("123456789"))
             .build());
   }
 
@@ -43,12 +42,10 @@
     ActionSketch sketch1 =
         ActionSketch.builder()
             .setTransitiveSourceHash(HashAndVersion.create(null, /*version=*/ Long.MAX_VALUE))
-            .setTransitiveActionLookupHash(null)
             .build();
     ActionSketch sketch2 =
         ActionSketch.builder()
             .setTransitiveSourceHash(HashAndVersion.create(null, /*version=*/ Long.MAX_VALUE))
-            .setTransitiveActionLookupHash(null)
             .build();
 
     assertThat(sketch1).isNotNull();