Skip to content

Conversation

@salvatore-campagna
Copy link
Contributor

Include date_histogram test on date_nanos field.
The test runs fine without any fix.

Issue: #39107

@salvatore-campagna salvatore-campagna changed the title test: date_histogram with time on date_nanos field test: date_histogram with time zone on date_nanos field Mar 21, 2022
@salvatore-campagna salvatore-campagna added >non-issue >test Issues or PRs that are addressing/adding tests :Analytics/Aggregations Aggregations labels Mar 21, 2022
@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Mar 21, 2022
@elasticmachine
Copy link
Collaborator

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

@salvatore-campagna
Copy link
Contributor Author

@elasticmachine update branch

@salvatore-campagna
Copy link
Contributor Author

@elasticmachine update branch

@not-napoleon
Copy link
Member

This test can just land in the existing 360_date_histogram.yml file

"date_histogram on date_nanos without timezone fixed interval":
- skip:
version: " - 8.1.99"
reason: bug fixed in 8.2.0
Copy link
Member

Choose a reason for hiding this comment

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

The linked issue lists #52016 as resolving this, which landed in 7.6.1. Let's confirm it works there or at least 7.7.0 and update the skip version appropriately. As noted in your other PR, I'm in favor of backporting these tests to 7.17.

Copy link
Member

@not-napoleon not-napoleon left a comment

Choose a reason for hiding this comment

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

LGTM

@salvatore-campagna
Copy link
Contributor Author

@elasticmachine update branch

@salvatore-campagna
Copy link
Contributor Author

@elasticmachine update branch

@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
7.17 Commit could not be cherrypicked due to conflicts
8.1

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 85149

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

Labels

:Analytics/Aggregations Aggregations >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v7.17.3 v8.1.3 v8.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants