Skip to content

Revert to SpotBugs 3.1.5#169

Merged
Ismael Juma (ijuma) merged 1 commit into5.1.xfrom
spotbugs-revert-3.1.5
Oct 21, 2018
Merged

Revert to SpotBugs 3.1.5#169
Ismael Juma (ijuma) merged 1 commit into5.1.xfrom
spotbugs-revert-3.1.5

Conversation

@ijuma
Copy link
Copy Markdown
Member

Newer versions break when processing Kafka code. A couple of examples:

Error processing opcode checkcast @ 63 in kafka.security.auth.SimpleAclAuthorizer.logMessage$1 : (Lorg.apache.kafka.common.security.auth.KafkaPrincipal;ZLkafka.security.auth.Operation;Lkafka.security.auth.Resource;Ljava.lang.String;)Ljava.lang.String;
Error processing opcode checkcast @ 31 in kafka.cluster.Broker.toString : ()Ljava.lang.String;
  java.lang.IllegalArgumentException: Unknown signature [Ljava/lang/Object; for number 4
    At edu.umd.cs.findbugs.OpcodeStack$Item.<init>(OpcodeStack.java:671)```

Newer versions break when processing Kafka code. A couple of examples:

Error processing opcode checkcast @ 63 in kafka.security.auth.SimpleAclAuthorizer.logMessage$1 : (Lorg.apache.kafka.common.security.auth.KafkaPrincipal;ZLkafka.security.auth.Operation;Lkafka.security.auth.Resource;Ljava.lang.String;)Ljava.lang.String;

Error processing opcode checkcast @ 31 in kafka.cluster.Broker.toString : ()Ljava.lang.String;
  java.lang.IllegalArgumentException: Unknown signature [Ljava/lang/Object; for number 4
    At edu.umd.cs.findbugs.OpcodeStack$Item.<init>(OpcodeStack.java:671)
@ijuma Ismael Juma (ijuma) merged commit 00494fe into 5.1.x Oct 21, 2018
@ijuma Ismael Juma (ijuma) deleted the spotbugs-revert-3.1.5 branch October 21, 2018 17:20
Ismael Juma (ijuma) added a commit that referenced this pull request Oct 21, 2018
* origin/5.1.x:
  Revert to SpotBugs 3.1.5 (#169)
Xavier Léauté (xvrl) added a commit that referenced this pull request Nov 12, 2018
- spotbugs 3.1.8 fixes the issue we saw in #169
- easymock 4.0.1 for JDK11 support
- powermock 2.0.0-rc3 for JDK11 support
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants