Uh oh!
There was an error while loading. Please reload this page.
Revert "[SPARK-29644][SQL] Corrected ShortType and ByteType mapping to SmallInt and TinyInt in JDBCUtils - #26583
Revert "[SPARK-29644][SQL] Corrected ShortType and ByteType mapping to SmallInt and TinyInt in JDBCUtils#26583shivsood wants to merge 1 commit into
Conversation
…o SmallInt and TinyInt in JDBCUtils" This reverts commit 32d44b1.
shivsood
commented
Nov 18, 2019
@dongjoon-hyun as FYI on revert for #26301 in master |
dongjoon-hyun
commented
Nov 18, 2019
Ur, do you want to revert fully? What I meant was the |
dongjoon-hyun
commented
Nov 18, 2019
cc @gatorsmile , @srowen |
shivsood
commented
Nov 18, 2019
@dongjoon-hyun 'll prefer a full revert to keep this clean and inline with what was done in 2.4. I will come back with PR for ShortType and ByteType separately as i get better grasp of the issue and fix. |
HyukjinKwon
commented
Nov 18, 2019
ok to test |
HyukjinKwon
commented
Nov 19, 2019
Either way seems fine to me. Let me merge this since the PR is open already. |
SparkQA
commented
Nov 19, 2019
Test build #114036 has finished for PR 26583 at commit
|
dongjoon-hyun
left a comment
There was a problem hiding this comment.
This is revert. Merged to master.
srowen
commented
Nov 19, 2019
Just for my info, all of the related changes in 2.4 are fully reverted now, so we can focus on master? |
dongjoon-hyun
commented
Nov 19, 2019
Yes. We checked the other |
This reverts commit f7e53865 i.e PR #26301 from master