Skip to content

Commit 80447b7

Browse files
authored
Merge branch 'main' into fix-fips-artifacts-in-release-tests
2 parents 21bc6f6 + 78b4168 commit 80447b7

File tree

3 files changed

+7
-2
lines changed
  • docs/changelog
  • x-pack/plugin
    • esql/compute/src/main/java/org/elasticsearch/compute/data
    • src/yamlRestTest/resources/rest-api-spec/test/esql

3 files changed

+7
-2
lines changed

docs/changelog/128960.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
pr: 128960
2+
summary: Throw ISE instead of IAE for illegal block in page
3+
area: ES|QL
4+
type: bug
5+
issues: []

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ private Page(boolean copyBlocks, int positionCount, Block[] blocks) {
8484
private Page(Page prev, Block[] toAdd) {
8585
for (Block block : toAdd) {
8686
if (prev.positionCount != block.getPositionCount()) {
87-
throw new IllegalArgumentException(
87+
throw new IllegalStateException(
8888
"Block [" + block + "] does not have same position count: " + block.getPositionCount() + " != " + prev.positionCount
8989
);
9090
}

x-pack/plugin/src/yamlRestTest/resources/rest-api-spec/test/esql/60_usage.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,7 @@ setup:
228228
- gt: {esql.functions.to_long: $functions_to_long}
229229
- match: {esql.functions.coalesce: $functions_coalesce}
230230
- gt: {esql.functions.categorize: $functions_categorize}
231-
- length: {esql.functions: 144} # check the "sister" test above for a likely update to the same esql.functions length check
231+
- length: {esql.functions: 145} # check the "sister" test above for a likely update to the same esql.functions length check
232232

233233
---
234234
took:

0 commit comments

Comments
 (0)