Lint

Author: afoucret

x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/rule/AbstractRuleTestCase.java

@@ -75,7 +75,7 @@ public String getWriteableName() {
         @Override
         public boolean equals(Object obj) {
             if (this == obj) return true;
-            if (!(obj instanceof TestNode)) return false;
+            if ((obj instanceof TestNode) == false) return false;
             TestNode other = (TestNode) obj;
             return value.equals(other.value) && children.equals(other.children);
         }

x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/rule/ParameterizedRuleExecutorTests.java

@@ -21,7 +21,7 @@ public class ParameterizedRuleExecutorTests extends AbstractRuleTestCase {
     static class TestParameterizedRuleExecutor extends ParameterizedRuleExecutor<TestNode, String> {
         public List<RuleExecutor.Batch<TestNode>> batches = new ArrayList<>();
 
-        public TestParameterizedRuleExecutor(String context) {
+        TestParameterizedRuleExecutor(String context) {
             super(context);
         }
 
@@ -34,7 +34,7 @@ public List<RuleExecutor.Batch<TestNode>> batches() {
     static class TestContextParameterizedRuleExecutor extends ParameterizedRuleExecutor<TestNode, TestContext> {
         public List<RuleExecutor.Batch<TestNode>> batches = new ArrayList<>();
 
-        public TestContextParameterizedRuleExecutor(TestContext context) {
+        TestContextParameterizedRuleExecutor(TestContext context) {
             super(context);
         }