Skip to content
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.

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

Update description of mandatory timestamp field #94983

Open
wants to merge 1 commit into
base: 8.7
Choose a base branch
from

Conversation

corradorai
Copy link

While experimenting with EQL I noticed it raised this error when the timestamp field was mapped as date with format yyyy-MM-dd HH:mm:ss:

search_phase_execution_exception: [parse_exception] Reason: failed to parse date field [1680434649000] with format [yyyy-MM-dd HH:mm:ss]

In particular, it happens when making a sequence query: https://www.elastic.co/guide/en/elasticsearch/reference/8.7/eql-syntax.html#eql-sequences

This gets solved when formatting the timestamp field as epoch_millis.

I'm not sure whether this is expected and fixing the docstring is enough or a bigfix ticket is needed.

@cla-checker-service
Copy link

❌ Author of the following commits did not sign a Contributor Agreement:
24d245b

Please, read and sign the above mentioned agreement if you want to contribute to this project

@github-actions
Copy link

github-actions bot commented Apr 3, 2023

Documentation preview:

@elasticsearchmachine elasticsearchmachine added v8.7.1 needs:triage Requires assignment of a team area label external-contributor Pull request authored by a developer outside the Elasticsearch team labels Apr 3, 2023
@pxsalehi pxsalehi added :Analytics/EQL EQL querying and removed needs:triage Requires assignment of a team area label labels Apr 4, 2023
@elasticsearchmachine elasticsearchmachine added the Team:QL (Deprecated) Meta label for query languages team label Apr 4, 2023
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-ql (Team:QL)

@pquentin pquentin added v8.7.2 and removed v8.7.1 labels Apr 12, 2023
@wchaparro wchaparro added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Jan 2, 2024
@elasticsearchmachine elasticsearchmachine removed the Team:QL (Deprecated) Meta label for query languages team label Jan 2, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/EQL EQL querying external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.7.2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants