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

JdbcCsvSpecIT#testCurrentDateFilter can fail with assertion error #65336

Closed
jtibshirani opened this issue Nov 21, 2020 · 7 comments · Fixed by #65348
Closed

JdbcCsvSpecIT#testCurrentDateFilter can fail with assertion error #65336

jtibshirani opened this issue Nov 21, 2020 · 7 comments · Fixed by #65348
Assignees
Labels
:Analytics/SQL SQL querying Team:QL (Deprecated) Meta label for query languages team >test-failure Triaged test failures from CI

Comments

@jtibshirani
Copy link
Contributor

Build scan: https://gradle-enterprise.elastic.co/s/4sbg4vwwi3wxc

Repro line:

./gradlew ':x-pack:plugin:sql:qa:server:security:without-ssl:integTest' --tests "org.elasticsearch.xpack.sql.qa.security.JdbcCsvSpecIT" -Dtests.method="test {date.testCurrentDateFilter}" -Dtests.seed=A44AF275CAF797A8 -Dtests.security.manager=true -Dtests.locale=ar-SA -Dtests.timezone=Egypt -Druntime.java=8

Reproduces locally?: yes

Applicable branches: 7.10, 7.x

Failure history: seems like it started failing earlier today. The first failure was at 12:05 UTC.

Failure excerpt:

 2> org.junit.ComparisonFailure: Different result for column [first_name], entry [9] expected:<B[ezalel]> but was:<B[ojan]>
        at __randomizedtesting.SeedInfo.seed([A44AF275CAF797A8:2C1ECDAF640BFA50]:0)
        at org.junit.Assert.assertEquals(Assert.java:115)
        at org.elasticsearch.xpack.sql.qa.jdbc.JdbcAssert.doAssertResultSetData(JdbcAssert.java:338)
        at org.elasticsearch.xpack.sql.qa.jdbc.JdbcAssert.assertResultSetData(JdbcAssert.java:222)
        at org.elasticsearch.xpack.sql.qa.jdbc.JdbcAssert.assertResultSets(JdbcAssert.java:105)
        at org.elasticsearch.xpack.sql.qa.jdbc.CsvSpecTestCase.assertResults(CsvSpecTestCase.java:55)
        at org.elasticsearch.xpack.sql.qa.jdbc.CsvSpecTestCase.executeAndAssert(CsvSpecTestCase.java:62)
        at org.elasticsearch.xpack.sql.qa.jdbc.CsvSpecTestCase.doTest(CsvSpecTestCase.java:48)
        at org.elasticsearch.xpack.sql.qa.jdbc.SpecBaseIntegrationTestCase.test(SpecBaseIntegrationTestCase.java:89)
@jtibshirani jtibshirani added >test-failure Triaged test failures from CI :Analytics/SQL SQL querying labels Nov 21, 2020
@elasticmachine elasticmachine added the Team:QL (Deprecated) Meta label for query languages team label Nov 21, 2020
@elasticmachine
Copy link
Collaborator

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

@jtibshirani
Copy link
Contributor Author

I'm not sure how to mute the test in a targeted way -- perhaps someone from @elastic/es-ql could take a look and mute if it's not immediately fixable?

@palesz
Copy link
Contributor

palesz commented Nov 21, 2020 via email

@original-brownbear
Copy link
Member

This also reproduces on master via:

./gradlew ':x-pack:plugin:sql:qa:server:security:with-ssl:integTest' --tests "org.elasticsearch.xpack.sql.qa.security.JdbcCsvSpecIT" -Dtests.seed=6EB27D271BC90F59 -Dtests.security.manager=true -Dtests.locale=ar-SD -Dtests.timezone=America/Merida -Druntime.java=11

original-brownbear added a commit to original-brownbear/elasticsearch that referenced this issue Nov 22, 2020
original-brownbear added a commit to original-brownbear/elasticsearch that referenced this issue Nov 22, 2020
original-brownbear added a commit to original-brownbear/elasticsearch that referenced this issue Nov 22, 2020
@original-brownbear
Copy link
Member

Muted in master, 7.x and 7.10 now

@hendrikmuhs
Copy link

@matriv
Copy link
Contributor

matriv commented Nov 23, 2020

@jtibshirani Just FYI, we have examples on how to mute SQL csv and sql tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/SQL SQL querying Team:QL (Deprecated) Meta label for query languages team >test-failure Triaged test failures from CI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants