Skip to content

Commit 5ea6cb5

Browse files
authored
ESQL: Fix and unmute BlockHashTests.test3BytesRefs() (#127830)
Fixes #127825 Fixes #127826 After merging #127582, there was a subtle change that made calls to `add(..., IntVectorBlock)` to directly call `add(..., IntVector)`. This triggered a code path in the test, throwing a NPE. Also, updated some autogenerated files that were merged before my other branch, and weren't updated
1 parent d0e3af7 commit 5ea6cb5

File tree

6 files changed

+262
-71
lines changed

6 files changed

+262
-71
lines changed

muted-tests.yml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -438,12 +438,6 @@ tests:
438438
- class: org.elasticsearch.xpack.remotecluster.RemoteClusterSecurityFcActionAuthorizationIT
439439
method: testIndicesPrivilegesAreEnforcedForCcrRestoreSessionActions
440440
issue: https://github.com/elastic/elasticsearch/issues/127782
441-
- class: org.elasticsearch.compute.aggregation.blockhash.BlockHashTests
442-
method: test3BytesRefs {forcePackedHash=true}
443-
issue: https://github.com/elastic/elasticsearch/issues/127825
444-
- class: org.elasticsearch.compute.aggregation.blockhash.BlockHashTests
445-
method: test3BytesRefs {forcePackedHash=false}
446-
issue: https://github.com/elastic/elasticsearch/issues/127826
447441
- class: org.elasticsearch.compute.aggregation.FilteredGroupingAggregatorFunctionTests
448442
method: testSimpleCircuitBreaking
449443
issue: https://github.com/elastic/elasticsearch/issues/127833

x-pack/plugin/esql/compute/src/main/generated/org/elasticsearch/compute/aggregation/FirstOverTimeDoubleGroupingAggregatorFunction.java

Lines changed: 65 additions & 16 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

x-pack/plugin/esql/compute/src/main/generated/org/elasticsearch/compute/aggregation/FirstOverTimeFloatGroupingAggregatorFunction.java

Lines changed: 65 additions & 16 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)