Skip to content

ESQL: Rebuild IsNotNull list when nullified by the JVM workaround - #149050

Merged
astefan merged 2 commits into
elastic:mainfrom
astefan:148944_fix
May 14, 2026
Merged

ESQL: Rebuild IsNotNull list when nullified by the JVM workaround#149050
astefan merged 2 commits into
elastic:mainfrom
astefan:148944_fix

Conversation

@astefan

@astefan astefan commented May 14, 2026

Copy link
Copy Markdown
Contributor

Workaround for #148944 by rebuilding the nullified Set when discovered as being surprisingly null. Followup of #148277

@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label May 14, 2026
@elasticsearchmachine

elasticsearchmachine commented May 14, 2026

Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine

elasticsearchmachine commented May 14, 2026

Copy link
Copy Markdown
Collaborator

Hi @astefan, I've created a changelog YAML for you.

@github-actions

github-actions Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

🔍 Preview links for changed docs

⏳ Building and deploying preview... View progress

This comment will be updated with preview links when the build is complete.

@github-actions

github-actions Bot commented May 14, 2026

Copy link
Copy Markdown
Contributor

ℹ️ Important: Docs version tagging

👋 Thanks for updating the docs! Just a friendly reminder that our docs are now cumulative. This means all 9.x versions are documented on the same page and published off of the main branch, instead of creating separate pages for each minor version.

We use applies_to tags to mark version-specific features and changes.

Expand for a quick overview

When to use applies_to tags:

✅ At the page level to indicate which products/deployments the content applies to (mandatory)
✅ When features change state (e.g. preview, ga) in a specific version
✅ When availability differs across deployments and environments

What NOT to do:

❌ Don't remove or replace information that applies to an older version
❌ Don't add new information that applies to a specific version without an applies_to tag
❌ Don't forget that applies_to tags can be used at the page, section, and inline level

🤔 Need help?

luigidellaquila
luigidellaquila approved these changes May 14, 2026
View reviewed changes

@luigidellaquila luigidellaquila left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @astefan, let's hope the JVM doesn't nullify it again and it doesn't trigger another NPE two lines below

@astefan
astefan merged commit 4c7c6c4 into elastic:main May 14, 2026
36 checks passed
@astefan
astefan deleted the 148944_fix branch May 14, 2026 10:47
@astefan astefan mentioned this pull request May 14, 2026
Closed
lukewhiting pushed a commit to lukewhiting/elasticsearch that referenced this pull request May 14, 2026
@astefan @lukewhiting
ESQL: Rebuild IsNotNull list when nullified by the JVM workaround (el…
841c136 
…astic#149050)
@kubasobon kubasobon mentioned this pull request May 15, 2026
Merged
10 tasks
jan-elastic added a commit that referenced this pull request Jul 31, 2026
@jan-elastic @astefan
9.4 backport propagate nullable fixes (#155596)
3ac2080 
* Helpful logging for NPE investigation (#147701)

* ESQL: Try to bypass jvm recompilation probably-bug (#148277)

* ESQL: Rebuild IsNotNull list when nullified by the JVM workaround (#149050)

* unmute tests

---------

Co-authored-by: Andrei Stefan <astefan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@luigidellaquila luigidellaquila luigidellaquila approved these changes

Assignees

No one assigned

Labels

:Analytics/ES|QL AKA ESQL >bug Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.5.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@astefan @elasticsearchmachine @luigidellaquila