Skip to content

Conversation

nrstott
Copy link

@nrstott nrstott commented Aug 1, 2016

Back-port JDBC column types to 1.x branch.

Author: Marc-André Tremblay <marcandre.tr@gmail.com>

This patch had conflicts when merged, resolved by
Committer: Josh Rosen <joshrosen@databricks.com>

Closes databricks#220 from marctrem/custom-jdbc-column-types.
@nrstott
Copy link
Author

nrstott commented Aug 1, 2016

Hope you don't mind the back-port request for this. Won't be able to update to Spark 2.x for a while. I understand the desire to freeze on new features but this is something that would be extremely useful so I thought I'd open this pull request.

@codecov-io
Copy link

codecov-io commented Aug 1, 2016

Current coverage is 89.45% (diff: 83.33%)

Merging #247 into branch-1.x will increase coverage by 0.03%

@@           branch-1.x       #247   diff @@
============================================
  Files              13         13          
  Lines             681        683     +2   
  Methods           596        592     -4   
  Messages            0          0          
  Branches           85         91     +6   
============================================
+ Hits              609        611     +2   
  Misses             72         72          
  Partials            0          0          

Powered by Codecov. Last update a1749cb...4a7fd8c

@JoshRosen
Copy link
Contributor

Let me go ahead and run the integration tests with this. If they pass then I'll merge this into the next 1.x release.

@JoshRosen
Copy link
Contributor

LGTM, so I'll merge this now.

@JoshRosen JoshRosen added this to the 1.0.1 milestone Aug 4, 2016
JoshRosen pushed a commit that referenced this pull request Aug 4, 2016
Back-port JDBC column types to 1.x branch.

Author: Marc-Andre Tremblay <marcandre.tr@gmail.com>

Closes #247 from nrstott/feature/custom-jdbc-column-types.
@JoshRosen JoshRosen closed this Aug 4, 2016
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