Skip to content

Conversation

@maropu
Copy link
Member

@maropu maropu commented May 21, 2021

What changes were proposed in this pull request?

This PR proposes to add a new GA test job for the docker integration tests.

Why are the changes needed?

To check the integration tests are healthy.

Does this PR introduce any user-facing change?

No, dev-only.

How was this patch tested?

GA passed.

@github-actions github-actions bot added the INFRA label May 21, 2021
@maropu
Copy link
Member Author

maropu commented May 21, 2021

It seems DB2KrbIntegrationSuite consistently fails, so I'm looking into this;

[info] DB2KrbIntegrationSuite:
[info] - SPARK-35226: JDBCOption should accept refreshKrb5Config parameter *** FAILED *** (2 seconds, 189 milliseconds)
[info]   com.ibm.db2.jcc.am.SqlSyntaxErrorException: DB2 SQL Error: SQLCODE=-104, SQLSTATE=42601, SQLERRMC=);ECT * FROM (SELECT 1;<table_expr>, DRIVER=4.26.14
[info]   at com.ibm.db2.jcc.am.b7.a(b7.java:810)
[info]   at com.ibm.db2.jcc.am.b7.a(b7.java:66)
[info]   at com.ibm.db2.jcc.am.b7.a(b7.java:140)
[info]   at com.ibm.db2.jcc.am.k4.c(k4.java:2824)

@SparkQA
Copy link

SparkQA commented May 21, 2021

Test build #138805 has finished for PR 32620 at commit ca6de3c.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@dongjoon-hyun dongjoon-hyun changed the title [WIP][SPARK-35481][SQL][TESTS] Add a new GA test job for the integration tests [WIP][SPARK-35481][SQL][TESTS] Add a new GA test job for the docker integration tests May 21, 2021
Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

I revised the title because we have two integration tests: JDBC and K8s. This one is for the first one.

@maropu
Copy link
Member Author

maropu commented May 21, 2021

Thank you, @dongjoon-hyun. right, the previous title was ambiguous.

@maropu maropu changed the title [WIP][SPARK-35481][SQL][TESTS] Add a new GA test job for the docker integration tests [WIP][SPARK-35483][SQL][TESTS] Add a new GA test job for the docker integration tests May 22, 2021
@maropu
Copy link
Member Author

maropu commented May 22, 2021

I used the wrong jira number, so I fixed it.

@maropu
Copy link
Member Author

maropu commented May 22, 2021

It seems @sarutak want to work on this, so I'll close this.

@maropu maropu closed this May 22, 2021
@maropu maropu deleted the IntegreTestJob branch June 23, 2021 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants