Skip to content

Conversation

@yaooqinn
Copy link
Member

What changes were proposed in this pull request?

After each database session is opened to the remote DB and before starting to read data, this option sessionInitStatement executes a custom SQL statement to do session initialization, this change enables this behavior for writing data.

Why are the changes needed?

Session initialization also need for writing data out of spark

Does this PR introduce any user-facing change?

no

How was this patch tested?

it

@SparkQA
Copy link

SparkQA commented Oct 12, 2019

Test build #111954 has finished for PR 26100 at commit 828d1ee.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Oct 12, 2019

Test build #111953 has finished for PR 26100 at commit 46e9081.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

@yaooqinn
Copy link
Member Author

retest this please

@SparkQA
Copy link

SparkQA commented Oct 12, 2019

Test build #111965 has finished for PR 26100 at commit 828d1ee.

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

@github-actions
Copy link

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!

@github-actions github-actions bot added the Stale label Jan 21, 2020
@github-actions github-actions bot closed this Jan 22, 2020
@msusicky
Copy link

Hi @yaooqinn , this is still a missing feature. Is there any reason not to continue with this pull request? (maybe more tests for more platform will be appreciated, I can help with that if it is needed)

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants