Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-10648] Oracle dialect to handle nonspecific numeric types #9495

Closed
wants to merge 4 commits into from
Closed

[SPARK-10648] Oracle dialect to handle nonspecific numeric types #9495

wants to merge 4 commits into from

Conversation

travishegner
Copy link
Contributor

This is the alternative/agreed upon solution to PR #8780.

Creating an OracleDialect to handle the nonspecific numeric types that can be defined in oracle.

@travishegner travishegner changed the title Oracle dialect to handle nonspecific numeric types [SPARK-10648] Oracle dialect to handle nonspecific numeric types Nov 5, 2015
@cloud-fan
Copy link
Contributor

ok to test

@SparkQA
Copy link

SparkQA commented Nov 5, 2015

Test build #45118 has finished for PR 9495 at commit 839bcb5.

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

* :: DeveloperApi ::
* Default Oracle dialect, mapping a nonspecific
* numeric type to a general decimal type.
* Solution by @cloud-fan and @bdolbeare (github.com)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@travishegner To provide more information, you can provide the links to their comments.

@SparkQA
Copy link

SparkQA commented Nov 5, 2015

Test build #45122 has finished for PR 9495 at commit a1370a7.

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

asfgit pushed a commit that referenced this pull request Nov 5, 2015
…umeric types

This backports #9495 to branch-1.4 (in case anyone needs this).

Author: Yin Huai <yhuai@databricks.com>

Closes #9498 from yhuai/OracleDialect-1.4.
asfgit pushed a commit that referenced this pull request Nov 5, 2015
…umeric types

This backports #9495 to branch-1.4 (in case anyone needs this).

Author: Yin Huai <yhuai@databricks.com>

Closes #9498 from yhuai/OracleDialect-1.4.

(cherry picked from commit 6c5e9a3)
Signed-off-by: Yin Huai <yhuai@databricks.com>
@yhuai
Copy link
Contributor

yhuai commented Nov 5, 2015

@travishegner I backported your change to branch 1.4 and branch 1.5 (see #9498) with a minor change on comments. Once you update your PR, I will merge it to master. Thanks!

@yhuai
Copy link
Contributor

yhuai commented Nov 5, 2015

I am merging it to master and branch 1.6. I will update the comments.

asfgit pushed a commit that referenced this pull request Nov 5, 2015
This is the alternative/agreed upon solution to PR #8780.

Creating an OracleDialect to handle the nonspecific numeric types that can be defined in oracle.

Author: Travis Hegner <thegner@trilliumit.com>

Closes #9495 from travishegner/OracleDialect.

(cherry picked from commit 14ee0f5)
Signed-off-by: Yin Huai <yhuai@databricks.com>
@asfgit asfgit closed this in 14ee0f5 Nov 5, 2015
@travishegner
Copy link
Contributor Author

Thanks @yhuai for taking care of this!

@dsdinter
Copy link

It seems this issue is in OJDBC and started to happen after Oracle 11g:
http://stackoverflow.com/questions/2133679/why-would-number-columns-scale-and-or-precision-differ-in-jdbc-from-oracle-10-t

@travishegner travishegner deleted the OracleDialect branch December 5, 2016 17:00
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.

5 participants