Skip to content

Conversation

@gatorsmile
Copy link
Member

What changes were proposed in this pull request?

This PR is to revert some code changes in the read path of #14377. The original fix is #17830

When merging this PR, please give the credit to @gaborfeher

How was this patch tested?

Added a test case to OracleIntegrationSuite.scala

@SparkQA
Copy link

SparkQA commented Jun 23, 2017

Test build #78538 has finished for PR 18408 at commit d8d4fad.

  • This patch fails PySpark pip packaging tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@gatorsmile
Copy link
Member Author

retest this please

@SparkQA
Copy link

SparkQA commented Jun 24, 2017

Test build #78546 has finished for PR 18408 at commit d8d4fad.

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

asfgit pushed a commit that referenced this pull request Jun 24, 2017
… in read path

## What changes were proposed in this pull request?

This PR is to revert some code changes in the read path of #14377. The original fix is #17830

When merging this PR, please give the credit to gaborfeher

## How was this patch tested?

Added a test case to OracleIntegrationSuite.scala

Author: Gabor Feher <[email protected]>
Author: gatorsmile <[email protected]>

Closes #18408 from gatorsmile/OracleType.

(cherry picked from commit b837bf9)
Signed-off-by: gatorsmile <[email protected]>
@gatorsmile
Copy link
Member Author

gatorsmile commented Jun 24, 2017

Thanks! Merging to master/2.2/2.1

@asfgit asfgit closed this in b837bf9 Jun 24, 2017
asfgit pushed a commit that referenced this pull request Jun 24, 2017
… in read path

This PR is to revert some code changes in the read path of #14377. The original fix is #17830

When merging this PR, please give the credit to gaborfeher

Added a test case to OracleIntegrationSuite.scala

Author: Gabor Feher <[email protected]>
Author: gatorsmile <[email protected]>

Closes #18408 from gatorsmile/OracleType.
robert3005 pushed a commit to palantir/spark that referenced this pull request Jun 29, 2017
… in read path

## What changes were proposed in this pull request?

This PR is to revert some code changes in the read path of apache#14377. The original fix is apache#17830

When merging this PR, please give the credit to gaborfeher

## How was this patch tested?

Added a test case to OracleIntegrationSuite.scala

Author: Gabor Feher <[email protected]>
Author: gatorsmile <[email protected]>

Closes apache#18408 from gatorsmile/OracleType.
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.

2 participants