remove NameID(long) ctr

ChrisHegarty · Sep 7, 2023 · 4d4d23a · 4d4d23a
1 parent 9a55717
commit 4d4d23a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/...plugin/esql/src/test/java/org/elasticsearch/xpack/esql/io/stream/PlanNamedTypesTests.java b/...plugin/esql/src/test/java/org/elasticsearch/xpack/esql/io/stream/PlanNamedTypesTests.java
@@ -169,7 +169,7 @@ public void testUnsupportedAttributeSimple() throws IOException {
             "foo",
             new UnsupportedEsField("foo", "keyword"),
             "field not supported",
-            new NameId(53)
+            new NameId()
         );
         BytesStreamOutput bso = new BytesStreamOutput();
         PlanStreamOutput out = new PlanStreamOutput(bso, planNameRegistry);
@@ -193,7 +193,7 @@ public void testFieldAttributeSimple() throws IOException {
             randomEsField(),
             null, // qualifier, can be null
             Nullability.TRUE,
-            new NameId(53),
+            new NameId(),
             true // synthetic
         );
         BytesStreamOutput bso = new BytesStreamOutput();

diff --git a/x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/expression/NameId.java b/x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/expression/NameId.java
@@ -26,10 +26,6 @@ public NameId() {
         this.id = COUNTER.incrementAndGet();
     }
 
-    public NameId(long id) {
-        this.id = id;
-    }
-
     @Override
     public int hashCode() {
         return Objects.hash(id);