[SPARK-34302][FOLLOWUP][SQL][TESTS] Update jdbc.v2.*IntegrationSuite

[dongjoon-hyun]

What changes were proposed in this pull request?

This PR aims to update JDBC v2 integration suite by adding catalogName.

Why are the changes needed?

To recover the integration test suite.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass the GitHub Action.

[cloud-fan]

thanks for fixing it!

BTW is there a way to capture it in PR builders? Otherwise, it's too easy to forget checking...

[dongjoon-hyun]

This seems to disabled by ENABLE_DOCKER_INTEGRATION_TESTS intentionally and designed to be turned on docker module change only at the original commit (2de19e4).

cc @sarutak and @HyukjinKwon .

[dongjoon-hyun]

I'll make a followup PR for SPARK-35483 to enable it for SQL part change, too.

[sarutak]

@dongjoon-hyun
I think It's because docker-integration-tests don't depend on sql module in dev/sparktestsupport/modules.py.

We intentionally let the dependency empty after the discussion.
If it's better to depends on sql module, I'll fix it.

[dongjoon-hyun]

Here is the follow-up for SPARK-35483.

• https://github.com/apache/spark/pull/33125/files

[dongjoon-hyun]

Docker integration test passed.

• https://github.com/dongjoon-hyun/spark/runs/2938755266?check_suite_focus=true

Merged to master. Thank you, @cloud-fan , @sarutak , @HyukjinKwon .

[imback82]

Thanks @dongjoon-hyun!