Skip to content

Conversation

@sureshthalamati
Copy link
Contributor

Currently credentials in JDBC URL/properties for jdbc data sources are included in the explain output. This fix removes credentials from the explain output and show only database table information.

Backporting fix to 1.6 from 2.0 as discussed in PR #10452

CC @marmbrus

@SparkQA
Copy link

SparkQA commented Jan 9, 2016

Test build #2355 has finished for PR 10669 at commit 212b4db.

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

@SparkQA
Copy link

SparkQA commented Jan 10, 2016

Test build #2356 has finished for PR 10669 at commit 212b4db.

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

@SparkQA
Copy link

SparkQA commented Jan 10, 2016

Test build #2357 has finished for PR 10669 at commit 212b4db.

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

@sureshthalamati
Copy link
Contributor Author

retest this please.

sparkR tests are passing in my branch. And also failed tests in the jenkins is not related to the changes in this PR.
@marmbrus

@gatorsmile
Copy link
Member

retest this please

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@sureshthalamati
Copy link
Contributor Author

It did not make it to 1.6 closing the PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants