Skip to content

Conversation

swallez
Copy link
Member

@swallez swallez commented Oct 14, 2025

When an ES|QL query is run with "profile": true, some additional properties are added after "values" in the response. The method EsqlAdapterBase.readFooter() was expecting the end of the response object, causing a parsing error.

This PR makes it more lenient: all remaining properties are ignored until the end of the object.

@l-trotta l-trotta merged commit 907ee93 into main Oct 20, 2025
11 checks passed
@l-trotta l-trotta deleted the lenient-sql-adapter branch October 20, 2025 12:40
github-actions bot pushed a commit that referenced this pull request Oct 20, 2025
Co-authored-by: Laura Trotta <153528055+l-trotta@users.noreply.github.com>
github-actions bot pushed a commit that referenced this pull request Oct 20, 2025
Co-authored-by: Laura Trotta <153528055+l-trotta@users.noreply.github.com>
github-actions bot pushed a commit that referenced this pull request Oct 20, 2025
Co-authored-by: Laura Trotta <153528055+l-trotta@users.noreply.github.com>
github-actions bot pushed a commit that referenced this pull request Oct 20, 2025
Co-authored-by: Laura Trotta <153528055+l-trotta@users.noreply.github.com>
l-trotta added a commit that referenced this pull request Oct 20, 2025
Co-authored-by: Sylvain Wallez <sylvain@elastic.co>
Co-authored-by: Laura Trotta <153528055+l-trotta@users.noreply.github.com>
l-trotta added a commit that referenced this pull request Oct 20, 2025
Co-authored-by: Sylvain Wallez <sylvain@elastic.co>
Co-authored-by: Laura Trotta <153528055+l-trotta@users.noreply.github.com>
l-trotta added a commit that referenced this pull request Oct 20, 2025
Co-authored-by: Sylvain Wallez <sylvain@elastic.co>
Co-authored-by: Laura Trotta <153528055+l-trotta@users.noreply.github.com>
l-trotta added a commit that referenced this pull request Oct 20, 2025
Co-authored-by: Sylvain Wallez <sylvain@elastic.co>
Co-authored-by: Laura Trotta <153528055+l-trotta@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants