Skip to content

ES|QL: Add generated evaluators for DateNanos conversion functions#111961

Merged
elasticsearchmachine merged 1 commit intoelastic:mainfrom
luigidellaquila:esql/from_datenanos_evaluators
Aug 19, 2024
Merged

ES|QL: Add generated evaluators for DateNanos conversion functions#111961
elasticsearchmachine merged 1 commit intoelastic:mainfrom
luigidellaquila:esql/from_datenanos_evaluators

Conversation

@luigidellaquila
Copy link
Copy Markdown
Member

These are generated files, resulting from this PR.

@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Aug 19, 2024
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

@luigidellaquila luigidellaquila added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Aug 19, 2024
Copy link
Copy Markdown
Contributor

@alex-spies alex-spies left a comment

Choose a reason for hiding this comment

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

LGTM, but as a side note: I think we started .gitignore-ing generated code instead; I wonder if we should include the evaluators, but definitely not in this PR.

@elasticsearchmachine elasticsearchmachine merged commit 06d09fc into elastic:main Aug 19, 2024
@luigidellaquila luigidellaquila deleted the esql/from_datenanos_evaluators branch August 19, 2024 09:01
weizijun added a commit to weizijun/elasticsearch that referenced this pull request Aug 19, 2024
* upstream/main:
  Fail `indexDocs()` on rejection (elastic#111962)
  Move repo analyzer to its own package (elastic#111963)
  Add generated evaluators for DateNanos conversion functions (elastic#111961)
  Clean the last traces from global retention in templates (elastic#111669)
  Fix known issue docs for elastic#111866 (elastic#111956)
  x-pack/plugin/otel: introduce x-pack-otel plugin (elastic#111091)
  Improve reaction to blob store corruptions (elastic#111954)
  Introduce `StreamingXContentResponse` (elastic#111933)
  Revert "Add 8.15.0 known issue for memory locking in Windows (elastic#111949)"
  Test get-snapshots API with missing details (elastic#111903)
  Add 8.15.0 known issue for memory locking in Windows (elastic#111949)

# Conflicts:
#	server/src/main/java/org/elasticsearch/TransportVersions.java
@nik9000
Copy link
Copy Markdown
Member

nik9000 commented Aug 20, 2024

LGTM, but as a side note: I think we started .gitignore-ing generated code instead; I wonder if we should include the evaluators, but definitely not in this PR.

I said I'd do it and never got to it actually. If you have time be my guest.

cbuescher pushed a commit to cbuescher/elasticsearch that referenced this pull request Sep 4, 2024
davidkyle pushed a commit to davidkyle/elasticsearch that referenced this pull request Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.16.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants