Skip to content

Commit f69a231

Browse files
committed
Update test expectations
1 parent a4aef2c commit f69a231

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/logical/local/ReplaceMissingFieldWithNull.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ private LogicalPlan missingToNull(LogicalPlan plan, Predicate<FieldAttribute> sh
6767
// Project[field1, field2, field3] <- keeps the ordering intact
6868
// \_Eval[field1 = null, field3 = null]
6969
// \_EsRelation[field2]
70-
// TODO: Test when there are 0 fields remaining
70+
// TODO: Double check that we test when there are 0 fields remaining
7171
List<Attribute> initialOutput = relation.output();
7272
List<Attribute> remainingFields = new ArrayList<>(initialOutput.size());
7373
Map<DataType, Alias> nullLiterals = Maps.newLinkedHashMapWithExpectedSize(DataType.types().size());

x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/optimizer/LocalPhysicalPlanOptimizerTests.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,8 @@
3838
import org.elasticsearch.xpack.esql.analysis.Verifier;
3939
import org.elasticsearch.xpack.esql.core.expression.Alias;
4040
import org.elasticsearch.xpack.esql.core.expression.Expressions;
41+
import org.elasticsearch.xpack.esql.core.expression.FieldAttribute;
4142
import org.elasticsearch.xpack.esql.core.expression.Literal;
42-
import org.elasticsearch.xpack.esql.core.expression.ReferenceAttribute;
4343
import org.elasticsearch.xpack.esql.core.tree.Source;
4444
import org.elasticsearch.xpack.esql.core.type.DataType;
4545
import org.elasticsearch.xpack.esql.core.type.EsField;
@@ -1340,9 +1340,9 @@ public void testMissingFieldsDoNotGetExtracted() {
13401340
)
13411341
);
13421342
// emp_no
1343-
assertThat(projections.get(1), instanceOf(ReferenceAttribute.class));
1343+
assertThat(projections.get(1), instanceOf(FieldAttribute.class));
13441344
// first_name
1345-
assertThat(projections.get(2), instanceOf(ReferenceAttribute.class));
1345+
assertThat(projections.get(2), instanceOf(FieldAttribute.class));
13461346

13471347
// last_name --> first_name
13481348
var nullAlias = Alias.unwrap(projections.get(8));

0 commit comments

Comments
 (0)