[SPARK-14504][SQL] Enable Oracle docker tests

[lresende]

What changes were proposed in this pull request?

Enable Oracle docker tests

How was this patch tested?

Existing tests

[lresende]

This PR is dependent on PR #9893

[SparkQA]

Test build #55419 has finished for PR 12270 at commit 9f65ab3.

• This patch fails to build.
• This patch merges cleanly.
• This patch adds no public classes.

[JoshRosen]

I just merged #9893 so this should be ready for a rebase.

[JoshRosen]

Ping RE: rebase.

[SparkQA]

Test build #56018 has finished for PR 12270 at commit f0790d5.

• This patch fails to build.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #56019 has finished for PR 12270 at commit a78c9a7.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #56021 has finished for PR 12270 at commit bb6658a.

• This patch fails to build.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #56024 has finished for PR 12270 at commit 71b99d8.

• This patch fails to build.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #56030 has finished for PR 12270 at commit 1490ca0.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #56031 has finished for PR 12270 at commit 5b17a11.

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

[SparkQA]

Test build #56042 has finished for PR 12270 at commit f281b43.

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

[SparkQA]

Test build #56062 has finished for PR 12270 at commit a5ccec6.

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

[lresende]

Jenkins, retest this

[lresende]

Jenkins, retest this please.

[JoshRosen]

Why this change?

[lresende]

The jar was getting a warning for checksum on maven build, and failing on sbt build... let me try removing it now that I explicitly pointed where the jar is for sbt, wich might make this change obsolete.

[SparkQA]

Test build #56087 has finished for PR 12270 at commit 9f92dc1.

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

[SparkQA]

Test build #56110 has finished for PR 12270 at commit 6b700e4.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[JoshRosen]

I'm a bit skeptical of some of the build hacks needed for this but out of pragmatism I'm going to merge this now and will just remove these tests later if this turns out to be unreliable.

[rxin]

These tests are too flaky. I've already seen a few failures.

We should disable them from the normal tests and maybe occasionally running them (via some trigger or just run it once before the release).

[rxin]

I just failed to build Spark locally once due to the docker oracle test.

[lresende]

@rxin, just trying to understand, is the oracle test the only one failing ? Or you are suggesting we move the whole docker based tests to a separate profile that would only run on Jenkins ?

[rxin]

I don't even think they should run on pull requests. Tests that require extensive external setup (or downloading things) in general are flaky.

[lresende]

@rxin Let me move them to a specific docker profile. But I would still run them on Jenkins, as the infrastructure is already setup there.

[rxin]

They have been flaky and causing other pull requests to fail. That's why we shouldn't run them on Jenkins by default.

[rxin]

We can run them via some trigger phrase though.

[lresende]

Ok, I will work with @JoshRosen on the trigger part.