ESQL: Remove obsolete comments on filtering (#104359)

Author: alex-spies

x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/BooleanArrayBlock.java

@@ -67,7 +67,6 @@ public boolean getBoolean(int valueIndex) {
 
     @Override
     public BooleanBlock filter(int... positions) {
-        // TODO use reference counting to share the vector
         try (var builder = blockFactory().newBooleanBlockBuilder(positions.length)) {
             for (int pos : positions) {
                 if (isNull(pos)) {

x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/BooleanBigArrayBlock.java

@@ -68,7 +68,6 @@ public boolean getBoolean(int valueIndex) {
 
     @Override
     public BooleanBlock filter(int... positions) {
-        // TODO use reference counting to share the vector
         try (var builder = blockFactory().newBooleanBlockBuilder(positions.length)) {
             for (int pos : positions) {
                 if (isNull(pos)) {

x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/BytesRefArrayBlock.java

@@ -70,7 +70,6 @@ public BytesRef getBytesRef(int valueIndex, BytesRef dest) {
 
     @Override
     public BytesRefBlock filter(int... positions) {
-        // TODO use reference counting to share the vector
         final BytesRef scratch = new BytesRef();
         try (var builder = blockFactory().newBytesRefBlockBuilder(positions.length)) {
             for (int pos : positions) {

x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/DoubleArrayBlock.java

@@ -67,7 +67,6 @@ public double getDouble(int valueIndex) {
 
     @Override
     public DoubleBlock filter(int... positions) {
-        // TODO use reference counting to share the vector
         try (var builder = blockFactory().newDoubleBlockBuilder(positions.length)) {
             for (int pos : positions) {
                 if (isNull(pos)) {

x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/DoubleBigArrayBlock.java

@@ -68,7 +68,6 @@ public double getDouble(int valueIndex) {
 
     @Override
     public DoubleBlock filter(int... positions) {
-        // TODO use reference counting to share the vector
         try (var builder = blockFactory().newDoubleBlockBuilder(positions.length)) {
             for (int pos : positions) {
                 if (isNull(pos)) {

x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/IntArrayBlock.java

@@ -67,7 +67,6 @@ public int getInt(int valueIndex) {
 
     @Override
     public IntBlock filter(int... positions) {
-        // TODO use reference counting to share the vector
         try (var builder = blockFactory().newIntBlockBuilder(positions.length)) {
             for (int pos : positions) {
                 if (isNull(pos)) {

x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/IntBigArrayBlock.java

@@ -68,7 +68,6 @@ public int getInt(int valueIndex) {
 
     @Override
     public IntBlock filter(int... positions) {
-        // TODO use reference counting to share the vector
         try (var builder = blockFactory().newIntBlockBuilder(positions.length)) {
             for (int pos : positions) {
                 if (isNull(pos)) {

x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/LongArrayBlock.java

@@ -67,7 +67,6 @@ public long getLong(int valueIndex) {
 
     @Override
     public LongBlock filter(int... positions) {
-        // TODO use reference counting to share the vector
         try (var builder = blockFactory().newLongBlockBuilder(positions.length)) {
             for (int pos : positions) {
                 if (isNull(pos)) {

x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/LongBigArrayBlock.java

@@ -68,7 +68,6 @@ public long getLong(int valueIndex) {
 
     @Override
     public LongBlock filter(int... positions) {
-        // TODO use reference counting to share the vector
         try (var builder = blockFactory().newLongBlockBuilder(positions.length)) {
             for (int pos : positions) {
                 if (isNull(pos)) {

x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/Block.java

@@ -107,8 +107,7 @@ public interface Block extends Accountable, BlockLoader.Block, NamedWriteable, R
     boolean mayHaveMultivaluedFields();
 
     /**
-     * Creates a new block that only exposes the positions provided. Materialization of the selected positions is avoided.
-     * The new block may hold a reference to this block, increasing this block's reference count.
+     * Creates a new block that only exposes the positions provided.
      * @param positions the positions to retain
      * @return a filtered block
      * TODO: pass BlockFactory