Started by an SCM change
Running as SYSTEM
[EnvInject] - Loading node environment variables.
Building remotely on ha:////4C8s9kS9T0JnOLz0YExs6DH29INIEFGDjE8FfR2V+TaWAAAAqh+LCAAAAAAAAP9b85aBtbiIQTGjNKU4P08vOT+vOD8nVc83PyU1x6OyILUoJzMv2y+/JJUBAhiZGBgqihhk0NSjKDWzXb3RdlLBUSYGJk8GtpzUvPSSDB8G5tKinBIGIZ+sxLJE/ZzEvHT94JKizLx0a6BxUmjGOUNodHsLgAyFEgY5/eT83ILSktQi/ZT85OzUIl0DAxOz3KSKdFOT1LxcABBOOEPTAAAAdocker-0046mbxg54enm on gcloud66 (bazel-debian bazel-chrome-69) in workspace /home/jenkins/workspace/plugin-gc-conductor-bazel-stable-2.16
Selected Git installation does not exist. Using Default
Wiping out workspace first.
Cloning the remote Git repository
Avoid fetching tags
Cloning repository https://gerrit.googlesource.com/a/gerrit
 > /usr/bin/git init /home/jenkins/workspace/plugin-gc-conductor-bazel-stable-2.16 # timeout=10
Fetching upstream changes from https://gerrit.googlesource.com/a/gerrit
 > /usr/bin/git --version # timeout=10
using GIT_ASKPASS to set credentials .netrc credentials for gerrit.googlesource.com
 > /usr/bin/git fetch --no-tags --force --progress -- https://gerrit.googlesource.com/a/gerrit +refs/heads/*:refs/remotes/gerrit/* # timeout=10
 > /usr/bin/git config remote.gerrit.url https://gerrit.googlesource.com/a/gerrit # timeout=10
 > /usr/bin/git config --add remote.gerrit.fetch +refs/heads/*:refs/remotes/gerrit/* # timeout=10
 > /usr/bin/git config remote.gerrit.url https://gerrit.googlesource.com/a/gerrit # timeout=10
Fetching upstream changes from https://gerrit.googlesource.com/a/gerrit
using GIT_ASKPASS to set credentials .netrc credentials for gerrit.googlesource.com
 > /usr/bin/git fetch --no-tags --force --progress -- https://gerrit.googlesource.com/a/gerrit +refs/heads/*:refs/remotes/gerrit/* # timeout=10
 > /usr/bin/git config remote.origin.url https://gerrit.googlesource.com/a/plugins/gc-conductor # timeout=10
Fetching upstream changes from https://gerrit.googlesource.com/a/plugins/gc-conductor
using GIT_ASKPASS to set credentials .netrc credentials for gerrit.googlesource.com
 > /usr/bin/git fetch --no-tags --force --progress -- https://gerrit.googlesource.com/a/plugins/gc-conductor +refs/heads/*:refs/remotes/origin/* # timeout=10
 > /usr/bin/git rev-parse origin/stable-2.16^{commit} # timeout=10
 > /usr/bin/git rev-parse refs/remotes/gerrit/origin/stable-2.16^{commit} # timeout=10
 > /usr/bin/git rev-parse refs/remotes/origin/stable-2.16^{commit} # timeout=10
 > /usr/bin/git rev-parse refs/remotes/origin/origin/stable-2.16^{commit} # timeout=10
Checking out Revision 554749b6aedf1bfbb5e86614ed49f808854a0f72 (origin/stable-2.16)
 > /usr/bin/git config core.sparsecheckout # timeout=10
 > /usr/bin/git checkout -f 554749b6aedf1bfbb5e86614ed49f808854a0f72 # timeout=10
Commit message: "Update JGit dependency to align with core's version"
 > /usr/bin/git rev-list --no-walk 60681a521cfccbfdf28b1f4cea5a140dbe4ebd93 # timeout=10
[plugin-gc-conductor-bazel-stable-2.16] $ /bin/bash -e /tmp/jenkins7589912698924021035.sh
[plugin-gc-conductor-bazel-stable-2.16] $ /bin/bash -e /tmp/jenkins2775400397319105239.sh
Java set to: /usr/lib/jvm/java-8-openjdk-amd64/bin/java
Previous HEAD position was 554749b6ae Update JGit dependency to align with core's version
Switched to a new branch 'stable-2.16'
Branch 'stable-2.16' set up to track remote branch 'stable-2.16' from 'gerrit'.
Submodule 'plugins/codemirror-editor' (https://gerrit.googlesource.com/a/plugins/codemirror-editor) registered for path 'plugins/codemirror-editor'
Submodule 'plugins/commit-message-length-validator' (https://gerrit.googlesource.com/a/plugins/commit-message-length-validator) registered for path 'plugins/commit-message-length-validator'
Submodule 'plugins/download-commands' (https://gerrit.googlesource.com/a/plugins/download-commands) registered for path 'plugins/download-commands'
Submodule 'plugins/hooks' (https://gerrit.googlesource.com/a/plugins/hooks) registered for path 'plugins/hooks'
Submodule 'plugins/replication' (https://gerrit.googlesource.com/a/plugins/replication) registered for path 'plugins/replication'
Submodule 'plugins/reviewnotes' (https://gerrit.googlesource.com/a/plugins/reviewnotes) registered for path 'plugins/reviewnotes'
Submodule 'plugins/singleusergroup' (https://gerrit.googlesource.com/a/plugins/singleusergroup) registered for path 'plugins/singleusergroup'
Cloning into '/home/jenkins/workspace/plugin-gc-conductor-bazel-stable-2.16/plugins/codemirror-editor'...
Cloning into '/home/jenkins/workspace/plugin-gc-conductor-bazel-stable-2.16/plugins/commit-message-length-validator'...
Cloning into '/home/jenkins/workspace/plugin-gc-conductor-bazel-stable-2.16/plugins/download-commands'...
Cloning into '/home/jenkins/workspace/plugin-gc-conductor-bazel-stable-2.16/plugins/hooks'...
Cloning into '/home/jenkins/workspace/plugin-gc-conductor-bazel-stable-2.16/plugins/replication'...
Cloning into '/home/jenkins/workspace/plugin-gc-conductor-bazel-stable-2.16/plugins/reviewnotes'...
Cloning into '/home/jenkins/workspace/plugin-gc-conductor-bazel-stable-2.16/plugins/singleusergroup'...
Submodule path 'plugins/codemirror-editor': checked out '59942b1adf1c949f3633f60ac42f67fae03b3255'
Submodule path 'plugins/commit-message-length-validator': checked out '556e427fd737744ce8a6a37b89fd427ae59bc8ea'
Submodule path 'plugins/download-commands': checked out '41c61bf8c1869bff4e0b436f69478c2137d0ca07'
Submodule path 'plugins/hooks': checked out 'e7769bd52c2b709d7712a386aa6cb96aec2c11a9'
Submodule path 'plugins/replication': checked out 'a6c453782de3f684a40a242bad9460faff75ffc3'
Submodule path 'plugins/reviewnotes': checked out 'ba3a422f1909f4970c563212d45cce70d494f60b'
Submodule path 'plugins/singleusergroup': checked out '2a38a8413c25fec910fb843932aa869647bdc1e7'
From https://gerrit.googlesource.com/a/plugins/gc-conductor
 * branch                  HEAD       -> FETCH_HEAD
openjdk full version "1.8.0_292-b10"
2021/09/10 16:12:21 Downloading https://releases.bazel.build/3.1.0/release/bazel-3.1.0-linux-x86_64...
Bazelisk version: v1.7.4
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
Build label: 3.1.0
Build target: bazel-out/k8-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Wed Apr 22 10:32:27 2020 (1587551547)
Build timestamp: 1587551547
Build timestamp as int: 1587551547
INFO: Invocation ID: 47218abb-5d49-47ae-82fb-beebbcf301e0
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Analyzing: target //plugins/gc-conductor:gc-conductor (1 packages loaded, 0 targets configured)
Analyzing: target //plugins/gc-conductor:gc-conductor (10 packages loaded, 7 targets configured)
Analyzing: target //plugins/gc-conductor:gc-conductor (10 packages loaded, 7 targets configured)
Analyzing: target //plugins/gc-conductor:gc-conductor (10 packages loaded, 7 targets configured)
Analyzing: target //plugins/gc-conductor:gc-conductor (10 packages loaded, 7 targets configured)
Analyzing: target //plugins/gc-conductor:gc-conductor (16 packages loaded, 103 targets configured)
Analyzing: target //plugins/gc-conductor:gc-conductor (132 packages loaded, 3037 targets configured)
Analyzing: target //plugins/gc-conductor:gc-conductor (181 packages loaded, 3146 targets configured)
Analyzing: target //plugins/gc-conductor:gc-conductor (197 packages loaded, 3751 targets configured)
Analyzing: target //plugins/gc-conductor:gc-conductor (198 packages loaded, 3790 targets configured)
INFO: Analyzed target //plugins/gc-conductor:gc-conductor (199 packages loaded, 4206 targets configured).
INFO: Found 1 target...
[0 / 54] [Prepa] BazelWorkspaceStatusAction stable-status.txt ... (2 actions, 0 running)
[55 / 341] Compiling external/com_google_protobuf/src/google/protobuf/extension_set.cc [for host]; 2s remote-cache, processwrapper-sandbox ... (31 actions, 30 running)
[76 / 341] Compiling external/com_google_protobuf/src/google/protobuf/generated_message_table_driven_lite.cc [for host]; 4s remote-cache, processwrapper-sandbox ... (31 actions, 30 running)
[95 / 341] Compiling external/com_google_protobuf/src/google/protobuf/compiler/php/php_generator.cc [for host]; 6s remote-cache, processwrapper-sandbox ... (31 actions, 30 running)
[117 / 341] Compiling external/com_google_protobuf/src/google/protobuf/compiler/command_line_interface.cc [for host]; 7s remote-cache, processwrapper-sandbox ... (31 actions, 30 running)
[144 / 341] Compiling external/com_google_protobuf/src/google/protobuf/compiler/csharp/csharp_message.cc [for host]; 4s remote-cache, processwrapper-sandbox ... (31 actions, 30 running)
[171 / 341] Compiling external/com_google_protobuf/src/google/protobuf/compiler/cpp/cpp_message.cc [for host]; 9s remote-cache, processwrapper-sandbox ... (31 actions, 30 running)
[196 / 341] Compiling external/com_google_protobuf/src/google/protobuf/descriptor.cc [for host]; 11s remote-cache, processwrapper-sandbox ... (31 actions, 30 running)
[319 / 365] Compiling external/com_google_protobuf/src/google/protobuf/descriptor.cc [for host]; 20s remote-cache, processwrapper-sandbox ... (11 actions, 10 running)
[345 / 365] Executing genrule //prolog:gerrit-prolog-common__pl2j; 2s remote-cache, processwrapper-sandbox ... (6 actions running)
[361 / 365] Building plugins/gc-conductor/libgc-conductor__plugin-class.jar (23 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 1s remote-cache, worker
Target //plugins/gc-conductor:gc-conductor up-to-date:
  bazel-bin/plugins/gc-conductor/gc-conductor.jar
INFO: Elapsed time: 85.082s, Critical Path: 40.33s
INFO: 348 processes: 342 processwrapper-sandbox, 6 worker.
INFO: Build completed successfully, 365 total actions
INFO: Build completed successfully, 365 total actions
INFO: Invocation ID: 659aa41e-e419-4844-8bd4-85bcf3c6e71a
Loading: 
Loading: 0 packages loaded
INFO: Build option --test_env has changed, discarding analysis cache.
Analyzing: 2 targets (4 packages loaded, 0 targets configured)
Analyzing: 2 targets (32 packages loaded, 641 targets configured)
Analyzing: 2 targets (95 packages loaded, 4795 targets configured)
INFO: Analyzed 2 targets (103 packages loaded, 4844 targets configured).
INFO: Found 2 test targets...
[0 / 1] [Prepa] BazelWorkspaceStatusAction stable-status.txt
[364 / 528] 1 / 2 tests; [Prepa] BazelWorkspaceStatusAction stable-status.txt ... (30 actions, 19 running)
[382 / 528] 1 / 2 tests; Building java/com/google/gerrit/util/ssl/libssl.jar (2 source files); 1s remote-cache, worker ... (32 actions, 22 running)
[387 / 528] 1 / 2 tests; Building java/com/google/gerrit/util/ssl/libssl.jar (2 source files); 2s remote-cache, worker ... (32 actions, 21 running)
[391 / 528] 1 / 2 tests; Building java/com/google/gerrit/util/ssl/libssl.jar (2 source files); 3s remote-cache, worker ... (32 actions, 21 running)
[392 / 528] 1 / 2 tests; Building java/com/google/gerrit/extensions/libclient-class.jar (24 source files); 4s remote-cache, worker ... (32 actions, 21 running)
[395 / 528] 1 / 2 tests; Building java/com/google/gerrit/extensions/libclient-class.jar (24 source files); 6s remote-cache, worker ... (32 actions, 20 running)
[404 / 528] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 7s remote-cache, worker ... (32 actions, 20 running)
INFO: From Building java/com/google/gerrit/util/cli/libcli.jar (7 source files):
java/com/google/gerrit/util/cli/CmdLineParser.java:333: warning: [BadAnnotationImplementation] Classes that implement Annotation must override equals and hashCode. Consider using AutoAnnotation instead of implementing Annotation by hand.
  private static class PrefixedOption implements Option {
                 ^
    (see https://errorprone.info/bugpattern/BadAnnotationImplementation)
java/com/google/gerrit/util/cli/CmdLineParser.java:511: warning: [BadAnnotationImplementation] Classes that implement Annotation must override equals and hashCode. Consider using AutoAnnotation instead of implementing Annotation by hand.
  private static class HelpOption implements Option, Setter<Boolean> {
                 ^
    (see https://errorprone.info/bugpattern/BadAnnotationImplementation)
java/com/google/gerrit/util/cli/CmdLineParser.java:512: warning: [ImmutableAnnotationChecker] annotations should be immutable: 'HelpOption' has non-final field 'value'
    private boolean value;
                    ^
    (see https://errorprone.info/bugpattern/ImmutableAnnotationChecker)
  Did you mean 'private final boolean value;'?
[406 / 528] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 9s remote-cache, worker ... (32 actions, 20 running)
[427 / 529] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 11s remote-cache, worker ... (32 actions, 21 running)
[431 / 529] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 13s remote-cache, worker ... (32 actions, 21 running)
[449 / 529] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 16s remote-cache, worker ... (32 actions, 20 running)
INFO: From Building java/com/google/gerrit/extensions/libapi.jar (329 source files):
java/com/google/gerrit/extensions/restapi/RestApiException.java:37: warning: [TypeParameterUnusedInFormals] Declaring a type parameter that is only used in the return type is a misuse of generics: operations on the type parameter are unchecked, it hides unsafe casts at invocations of the method, and it interacts badly with method overload resolution.
  public <T extends RestApiException> T caching(CacheControl c) {
                                        ^
    (see https://errorprone.info/bugpattern/TypeParameterUnusedInFormals)
[455 / 529] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 19s remote-cache, worker ... (32 actions, 24 running)
[468 / 529] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 22s remote-cache, worker ... (32 actions, 24 running)
INFO: From Building java/com/google/gerrit/pgm/init/api/libapi.jar (14 source files):
java/com/google/gerrit/pgm/init/api/Section.java:131: warning: [GetClassOnEnum] Calling getClass() on an enum may return a subclass of the enum type
    E allowedValues = (E) EnumSet.allOf(defValue.getClass());
                                                         ^
    (see https://errorprone.info/bugpattern/GetClassOnEnum)
  Did you mean 'E allowedValues = (E) EnumSet.allOf(defValue.getDeclaringClass());'?
[476 / 529] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 26s remote-cache, worker ... (32 actions, 21 running)
[483 / 529] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 30s remote-cache, worker ... (32 actions, 22 running)
[488 / 529] 1 / 2 tests; Building external/com_google_protobuf/libprotobuf_java.jar (123 source files, 1 source jar); 35s remote-cache, worker ... (32 actions, 20 running)
INFO: From Building java/com/google/gerrit/sshd/libsshd.jar (101 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor):
java/com/google/gerrit/sshd/commands/ApproveOption.java:32: warning: [BadAnnotationImplementation] Classes that implement Annotation must override equals and hashCode. Consider using AutoAnnotation instead of implementing Annotation by hand.
final class ApproveOption implements Option, Setter<Short> {
      ^
    (see https://errorprone.info/bugpattern/BadAnnotationImplementation)
java/com/google/gerrit/sshd/commands/ApproveOption.java:35: warning: [ImmutableAnnotationChecker] annotations should be immutable: 'ApproveOption' has field 'type' of type 'com.google.gerrit.common.data.LabelType', the declaration of type 'com.google.gerrit.common.data.LabelType' is not annotated with @com.google.errorprone.annotations.Immutable
  private final LabelType type;
                          ^
    (see https://errorprone.info/bugpattern/ImmutableAnnotationChecker)
java/com/google/gerrit/sshd/commands/ApproveOption.java:37: warning: [ImmutableAnnotationChecker] annotations should be immutable: 'ApproveOption' has non-final field 'value'
  private Short value;
                ^
    (see https://errorprone.info/bugpattern/ImmutableAnnotationChecker)
  Did you mean 'private final Short value;'?
java/com/google/gerrit/sshd/commands/StreamEvents.java:284: warning: [SynchronizeOnNonFinalField] Synchronizing on non-final fields is not safe: if the field is ever updated, different threads may end up locking on different objects.
      synchronized (stdout) {
                   ^
    (see https://errorprone.info/bugpattern/SynchronizeOnNonFinalField)
java/com/google/gerrit/sshd/commands/StreamEvents.java:291: warning: [SynchronizeOnNonFinalField] Synchronizing on non-final fields is not safe: if the field is ever updated, different threads may end up locking on different objects.
    synchronized (stdout) {
                 ^
    (see https://errorprone.info/bugpattern/SynchronizeOnNonFinalField)
java/com/google/gerrit/sshd/commands/UploadArchive.java:142: warning: [ReferenceEquality] Comparison using reference equality instead of value equality
      if (s == PacketLineIn.END) {
            ^
    (see https://errorprone.info/bugpattern/ReferenceEquality)
  Did you mean 'if (Objects.equals(s, PacketLineIn.END)) {' or 'if (s.equals(PacketLineIn.END)) {'?
java/com/google/gerrit/sshd/commands/UploadArchive.java:142: warning: [StringEquality] String comparison using reference equality instead of value equality
      if (s == PacketLineIn.END) {
            ^
    (see https://errorprone.info/bugpattern/StringEquality)
  Did you mean 'if (Objects.equals(s, PacketLineIn.END)) {' or 'if (s.equals(PacketLineIn.END)) {'?
[494 / 529] 1 / 2 tests; Building java/com/google/gerrit/server/libserver-class.jar (990 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 20s remote-cache, worker ... (32 actions, 21 running)
INFO: From Building java/com/google/gerrit/httpd/libhttpd-class.jar (115 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor):
java/com/google/gerrit/httpd/HttpServletResponseRecorder.java:74: warning: [MissingOverride] getStatus overrides method in HttpServletResponseWrapper; expected @Override
  public int getStatus() {
             ^
    (see https://errorprone.info/bugpattern/MissingOverride)
  Did you mean '@Override @SuppressWarnings("all")'?
java/com/google/gerrit/httpd/rpc/AuditedHttpServletResponse.java:29: warning: [MissingOverride] getStatus overrides method in HttpServletResponseWrapper; expected @Override
  public int getStatus() {
             ^
    (see https://errorprone.info/bugpattern/MissingOverride)
  Did you mean '@Override @SuppressWarnings("all")'?
[513 / 529] 1 / 2 tests; Building java/com/google/gerrit/server/libserver-class.jar (990 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 26s remote-cache, worker ... (18 actions, 15 running)
INFO: From Building java/com/google/gerrit/lucene/liblucene.jar (12 source files):
java/com/google/gerrit/lucene/WrappableSearcherManager.java:191: warning: [ReferenceEquality] Comparison using reference equality instead of value equality
        if (unwrapped == reader) {
                      ^
    (see https://errorprone.info/bugpattern/ReferenceEquality)
  Did you mean 'if (Objects.equals(unwrapped, reader)) {' or 'if (unwrapped.equals(reader)) {'?
java/com/google/gerrit/lucene/WrappableSearcherManager.java:202: warning: [ReferenceEquality] Comparison using reference equality instead of value equality
      if (unwrapped != reader) {
                    ^
    (see https://errorprone.info/bugpattern/ReferenceEquality)
  Did you mean 'if (!Objects.equals(unwrapped, reader)) {' or 'if (!unwrapped.equals(reader)) {'?
INFO: From Building java/com/google/gerrit/acceptance/libframework-lib.jar (61 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor):
java/com/google/gerrit/acceptance/GerritServer.java:461: warning: [TypeParameterUnusedInFormals] Declaring a type parameter that is only used in the return type is a misuse of generics: operations on the type parameter are unchecked, it hides unsafe casts at invocations of the method, and it interacts badly with method overload resolution.
  private static <T> T get(Object obj, String field)
                       ^
    (see https://errorprone.info/bugpattern/TypeParameterUnusedInFormals)
java/com/google/gerrit/acceptance/ConfigAnnotationParser.java:48: warning: [BadAnnotationImplementation] Classes that implement Annotation must override equals and hashCode. Consider using AutoAnnotation instead of implementing Annotation by hand.
  static class GlobalPluginConfigToGerritConfig implements GerritConfig {
         ^
    (see https://errorprone.info/bugpattern/BadAnnotationImplementation)
[527 / 529] 1 / 2 tests; Building java/com/google/gerrit/server/libserver-class.jar (990 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor); 34s remote-cache, worker
INFO: From Building java/com/google/gerrit/server/libserver-class.jar (990 source files) and running annotation processors (AutoAnnotationProcessor, AutoValueProcessor):
java/com/google/gerrit/server/notedb/ReviewerStateInternal.java:53: warning: [ImmutableEnumChecker] enums should be immutable: 'ReviewerStateInternal' has field 'footerKey' of type 'org.eclipse.jgit.revwalk.FooterKey', the declaration of type 'org.eclipse.jgit.revwalk.FooterKey' is not annotated with @com.google.errorprone.annotations.Immutable
  private final FooterKey footerKey;
                          ^
    (see https://errorprone.info/bugpattern/ImmutableEnumChecker)
java/com/google/gerrit/server/index/change/ChangeIndexRewriter.java:158: warning: [ReferenceEquality] Comparison using reference equality instead of value equality
    if (in == out || out instanceof IndexPredicate) {
           ^
    (see https://errorprone.info/bugpattern/ReferenceEquality)
  Did you mean 'if (Objects.equals(in, out) || out instanceof IndexPredicate) {' or 'if (in.equals(out) || out instanceof IndexPredicate) {'?
java/com/google/gerrit/server/index/change/ChangeIndexRewriter.java:210: warning: [ReferenceEquality] Comparison using reference equality instead of value equality
      if (nc == c) {
             ^
    (see https://errorprone.info/bugpattern/ReferenceEquality)
  Did you mean 'if (Objects.equals(nc, c)) {' or 'if (nc.equals(c)) {'?
java/com/google/gerrit/server/util/IdGenerator.java:48: warning: [NarrowingCompoundAssignment] Compound assignments from int to short hide lossy casts
    v0 += ((v1 << 2) + 0 ^ v1) + (salt ^ (v1 >>> 3)) + 1;
       ^
    (see https://errorprone.info/bugpattern/NarrowingCompoundAssignment)
  Did you mean 'v0 = (short) (v0 + (((v1 << 2) + 0 ^ v1) + (salt ^ (v1 >>> 3)) + 1));'?
java/com/google/gerrit/server/util/IdGenerator.java:49: warning: [NarrowingCompoundAssignment] Compound assignments from int to short hide lossy casts
    v1 += ((v0 << 2) + 2 ^ v0) + (salt ^ (v0 >>> 3)) + 3;
       ^
    (see https://errorprone.info/bugpattern/NarrowingCompoundAssignment)
  Did you mean 'v1 = (short) (v1 + (((v0 << 2) + 2 ^ v0) + (salt ^ (v0 >>> 3)) + 3));'?
java/com/google/gerrit/server/util/IdGenerator.java:57: warning: [NarrowingCompoundAssignment] Compound assignments from int to short hide lossy casts
    v1 -= ((v0 << 2) + 2 ^ v0) + (salt ^ (v0 >>> 3)) + 3;
       ^
    (see https://errorprone.info/bugpattern/NarrowingCompoundAssignment)
  Did you mean 'v1 = (short) (v1 - (((v0 << 2) + 2 ^ v0) + (salt ^ (v0 >>> 3)) + 3));'?
java/com/google/gerrit/server/util/IdGenerator.java:58: warning: [NarrowingCompoundAssignment] Compound assignments from int to short hide lossy casts
    v0 -= ((v1 << 2) + 0 ^ v1) + (salt ^ (v1 >>> 3)) + 1;
       ^
    (see https://errorprone.info/bugpattern/NarrowingCompoundAssignment)
  Did you mean 'v0 = (short) (v0 - (((v1 << 2) + 0 ^ v1) + (salt ^ (v1 >>> 3)) + 1));'?
java/com/google/gerrit/server/change/ArchiveFormat.java:38: warning: [ImmutableEnumChecker] enums should be immutable: 'ArchiveFormat' has field 'format' of type 'org.eclipse.jgit.api.ArchiveCommand.Format<?>', the declaration of type 'org.eclipse.jgit.api.ArchiveCommand.Format<?>' is not annotated with @com.google.errorprone.annotations.Immutable
  private final ArchiveCommand.Format<?> format;
                                         ^
    (see https://errorprone.info/bugpattern/ImmutableEnumChecker)
java/com/google/gerrit/server/change/ChangeKindCacheImpl.java:386: warning: [ReferenceEquality] Comparison using reference equality instead of value equality
              && (ps.getId().get() > priorPs.getId().get() || priorPs == patch)) {
                                                                      ^
    (see https://errorprone.info/bugpattern/ReferenceEquality)
  Did you mean '&& (ps.getId().get() > priorPs.getId().get() || priorPs.equals(patch))) {'?
java/com/google/gerrit/server/change/ChangeKindCacheImpl.java:396: warning: [ReferenceEquality] Comparison using reference equality instead of value equality
        if (priorPs != patch) {
                    ^
    (see https://errorprone.info/bugpattern/ReferenceEquality)
  Did you mean 'if (!priorPs.equals(patch)) {'?
java/com/google/gerrit/server/index/AbstractIndexModule.java:78: warning: [TypeParameterUnusedInFormals] Declaring a type parameter that is only used in the return type is a misuse of generics: operations on the type parameter are unchecked, it hides unsafe casts at invocations of the method, and it interacts badly with method overload resolution.
  private static <T> T createDummyIndexFactory(Schema<?> schema) {
                       ^
    (see https://errorprone.info/bugpattern/TypeParameterUnusedInFormals)
java/com/google/gerrit/server/notedb/NotesMigrationState.java:60: warning: [ImmutableEnumChecker] enums should be immutable: 'NotesMigrationState' has field 'snapshot' of type 'com.google.gerrit.server.notedb.NotesMigration.Snapshot', the declaration of type 'com.google.gerrit.server.notedb.NotesMigration.Snapshot' is not annotated with @com.google.errorprone.annotations.Immutable
  private final Snapshot snapshot;
                         ^
    (see https://errorprone.info/bugpattern/ImmutableEnumChecker)
java/com/google/gerrit/server/patch/PatchScriptBuilder.java:504: warning: [ReferenceEquality] Comparison using reference equality instead of value equality
              && (aId == within || within.equals(aId))) {
                      ^
    (see https://errorprone.info/bugpattern/ReferenceEquality)
  Did you mean '&& (Objects.equals(aId, within) || within.equals(aId))) {' or '&& (aId.equals(within) || within.equals(aId))) {'?
java/com/google/gerrit/server/patch/PatchScriptBuilder.java:524: warning: [ReferenceEquality] Comparison using reference equality instead of value equality
              && (aId == within || within.equals(aId))) {
                      ^
    (see https://errorprone.info/bugpattern/ReferenceEquality)
  Did you mean '&& (Objects.equals(aId, within) || within.equals(aId))) {' or '&& (aId.equals(within) || within.equals(aId))) {'?
[529 / 530] 1 / 2 tests; Testing //plugins/gc-conductor:gc-conductor_tests; 3s remote-cache, processwrapper-sandbox
[529 / 530] 1 / 2 tests; Testing //plugins/gc-conductor:gc-conductor_tests; 24s remote-cache, processwrapper-sandbox
[529 / 530] 1 / 2 tests; Testing //plugins/gc-conductor:gc-conductor_tests; 41s remote-cache, processwrapper-sandbox
[529 / 530] 2 / 2 tests; Testing //plugins/gc-conductor:gc-conductor_tests; 65s remote-cache, processwrapper-sandbox
INFO: Elapsed time: 130.677s, Critical Path: 112.39s
INFO: 165 processes: 99 processwrapper-sandbox, 66 worker.
INFO: Build completed successfully, 172 total actions
//plugins/gc-conductor:gc-conductor_tests                                PASSED in 65.4s
//tools/bzl:always_pass_test                                             PASSED in 0.1s

Executed 2 out of 2 tests: 2 tests pass.
There were tests whose specified size is too big. Use the --test_verbose_timeout_warnings command line option to see which ones these are.
INFO: Build completed successfully, 172 total actions
Sep 10, 2021 4:16:02 PM java.util.jar.Attributes read
WARNING: Duplicate name in Manifest: Implementation-Version.
Ensure that the manifest does not have duplicate entries, and
that blank lines separate individual sections in both your
manifest and in the META-INF/MANIFEST.MF entry in the jar file.
[plugin-gc-conductor-bazel-stable-2.16] $ /bin/bash -e /tmp/jenkins4474886308810720014.sh
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100  1859    0  1859    0     0  21367      0 --:--:-- --:--:-- --:--:-- 21367
Archiving artifacts
Finished: SUCCESS