Skip to content

Updated documentation for ESQL limitations on counter #116144

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 0 commits into from
Closed

Updated documentation for ESQL limitations on counter #116144

wants to merge 0 commits into from

Conversation

emilywcai
Copy link

As a fix for this issue, a note has been added to the ESQL Limitations doc, where counter is listed under unsupported types. It has been specified that there is limited support via explicit casting for these 3 types: counter_long, counter_integer, and counter_double.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 3, 2024

Documentation preview:

@elasticsearchmachine elasticsearchmachine added v9.0.0 needs:triage Requires assignment of a team area label external-contributor Pull request authored by a developer outside the Elasticsearch team labels Nov 3, 2024
@emilywcai emilywcai mentioned this pull request Nov 3, 2024
Closed
@gbanasiak gbanasiak added the :Analytics/ES|QL AKA ESQL label Nov 22, 2024
@elasticsearchmachine elasticsearchmachine added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) and removed needs:triage Requires assignment of a team area label labels Nov 22, 2024
@elasticsearchmachine
Copy link
Collaborator

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

@alex-spies alex-spies added the :StorageEngine/TSDB You know, for Metrics label Feb 7, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-storage-engine (Team:StorageEngine)

@alex-spies
Copy link
Contributor

This seems to be TSDB-related. @elastic/es-storage-engine , I hope it's correct to label this with your team!

@alex-spies alex-spies removed the :Analytics/ES|QL AKA ESQL label Feb 7, 2025
@elasticsearchmachine elasticsearchmachine removed the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Feb 7, 2025
@kkrik-es kkrik-es requested a review from dnhatn February 10, 2025 06:18
martijnvg
martijnvg approved these changes Feb 10, 2025
View reviewed changes
Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions GitHub Actions
Copy link
Contributor

Warning

It looks like this PR modifies one or more .asciidoc files. These files are being migrated to Markdown, and any changes merged now will be lost. See the migration guide for details.

@dnhatn dnhatn added v9.0.0 v8.19.0 v8.18.1 >docs General docs changes auto-backport Automatically create backport pull requests when merged labels Feb 11, 2025
@dnhatn dnhatn self-assigned this Feb 11, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@elasticsearchmachine elasticsearchmachine added the Team:Docs Meta label for docs team label Feb 11, 2025
@github-actions GitHub Actions
Copy link
Contributor

Warning

It looks like this PR modifies one or more .asciidoc files. These files are being migrated to Markdown, and any changes merged now will be lost. See the migration guide for details.

@leemthompo
Copy link
Contributor

👋 Please rebase to target only 8.x branches and create a follow-up issue in docs-content repo to update the new 9.0 docs in markdown post documentation freeze 🙏

All changes to asciidoc files on main/9.x branches will be lost

@wchaparro
Copy link
Member

@martijnvg still in progress?

@pabloem pabloem closed this Jun 5, 2025
@pabloem
Copy link
Contributor

pabloem commented Jun 5, 2025

hi @emilywcai - looks like I messed up your PR trying to rebase it. Would you be willing to open a new one? If not, I can open a PR myself with the changes you proposed. WDYT?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@martijnvg martijnvg martijnvg approved these changes

@dnhatn dnhatn Awaiting requested review from dnhatn

Assignees

@dnhatn dnhatn

Labels

auto-backport Automatically create backport pull requests when merged >docs General docs changes external-contributor Pull request authored by a developer outside the Elasticsearch team :StorageEngine/TSDB You know, for Metrics Team:Docs Meta label for docs team Team:StorageEngine v8.18.3 v8.19.0 v9.0.0 v9.1.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@emilywcai @elasticsearchmachine @alex-spies @leemthompo @wchaparro @pabloem @martijnvg @dnhatn @gbanasiak