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.

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

[CI] SearchFieldsIT testDocValueFields failure #31992

Closed
martijnvg opened this issue Jul 12, 2018 · 1 comment
Closed

[CI] SearchFieldsIT testDocValueFields failure #31992

martijnvg opened this issue Jul 12, 2018 · 1 comment
Assignees
@spinscale
Labels
:Search/Search Search-related issues that do not fall into other categories >test-failure Triaged test failures from CI

Comments

@martijnvg
Copy link
Member

Failure:

09:47:47    > Throwable #1: java.lang.ClassCastException: org.joda.time.MutableDateTime cannot be cast to org.joda.time.DateTime
09:47:47    > 	at __randomizedtesting.SeedInfo.seed([46906270EEFFE0CB:CA6A1FC86A640665]:0)
09:47:47    > 	at org.elasticsearch.search.fields.SearchFieldsIT.testDocValueFields(SearchFieldsIT.java:806)
09:47:47    > 	at java.lang.Thread.run(Thread.java:748)

Failure reproduces:

./gradlew :server:integTest -Dtests.seed=46906270EEFFE0CB -Dtests.class=org.elasticsearch.search.fields.SearchFieldsIT -Dtests.method="testDocValueFields" -Dtests.security.manager=true -Dtests.locale=is-IS -Dtests.timezone=Asia/Jerusalem
09:47:47   1> [2018-07-12T12:47:47,113][INFO ][o.e.c.m.MetaDataDeleteIndexService] [node_sm0] [test/j69M4xO1Qvq58L5XmKFb

Build url: https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+intake/2305/console
@martijnvg martijnvg added :Search/Search Search-related issues that do not fall into other categories >test-failure Triaged test failures from CI labels Jul 12, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search-aggs

spinscale added a commit that referenced this issue Jul 12, 2018
@spinscale
Tests: Mute SearchFieldsIT.testDocValueFields()
e3707ef 
Relates #31992
spinscale added a commit to spinscale/elasticsearch that referenced this issue Jul 12, 2018
@spinscale
Tests: Fix SearchFieldsIT.testDocValueFields
727471b 
This test produced different implementations of joda time classes,
depending on if the data was serialized or not (DateTime vs
MutableDateTime). This now uses a common base class to extract the
milliseconds from the data.

Closes elastic#31992
@spinscale spinscale mentioned this issue Jul 12, 2018
Merged
spinscale added a commit that referenced this issue Jul 12, 2018
@spinscale
Tests: Fix SearchFieldsIT.testDocValueFields (#31995)
0b7e7be 
This test produced different implementations of joda time classes,
depending on if the data was serialized or not (DateTime vs
MutableDateTime). This now uses a common base class to extract the
milliseconds from the data.

Closes #31992
spinscale added a commit to spinscale/elasticsearch that referenced this issue Jul 16, 2018
@spinscale
Tests: Fix SearchFieldsIT.testDocValueFields
2c3040f 
This test produced different implementations of joda time classes,
depending on if the data was serialized or not (DateTime vs
MutableDateTime). This now uses a common base class to extract the
milliseconds from the data.

Closes elastic#31992
rjernst pushed a commit that referenced this issue Jul 16, 2018
@spinscale @rjernst
Core: Backport java time date formatters (#31997)
a87ae91 
* Date: Add DateFormatters class that uses java.time (#31856)

A newly added class called DateFormatters now contains java.time based
builders for dates, which also intends to be fully backwards compatible,
when the name based date formatters are picked. Also a new class named
CompoundDateTimeFormatter for being able to parse multiple different
formats has been added.

A duelling test class has been added that ensures the same dates when
parsing java or joda time formatted dates for the name based dates.

Note, that java.time and joda time are not fully backwards compatible,
which also means that old formats will currently not work with this
setup.

* Tests: Remove use of joda time in some tests (#31922)

This also extends the dateformatters test to ensure that the printers
are acting the same in java time and joda time.

* Tests: Fix SearchFieldsIT.testDocValueFields

This test produced different implementations of joda time classes,
depending on if the data was serialized or not (DateTime vs
MutableDateTime). This now uses a common base class to extract the
milliseconds from the data.

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

@spinscale spinscale

Labels
:Search/Search Search-related issues that do not fall into other categories >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@martijnvg @spinscale @elasticmachine