Resolve Hive API deprecation warnings

lucidworks / hive-solr

Code to index Hive tables to Solr and Solr indexes to Hive

Apache License 2.0

47 stars 34 forks source link

Resolve Hive API deprecation warnings #45

Closed gerlowskija closed 6 years ago

gerlowskija commented 6 years ago

Prior to this commit, several classes used deprecated Hive APIs, that ended up being removed in the Hive 2.x release line. This commit fixes a few of these warnings, to clear the way for use of Hive 2.x releases coming soon. Specifically, the changes involved:

changing references to "Constants" to its successor "serdeConstants"
using the "AbstractSerDe" class as a supertype, instead of the "SerDe" interface

ctargett commented 6 years ago

With this PR, would it be impossible for the SerDe to work with Hive 1.x? Just curious.

gerlowskija commented 6 years ago

Nope. This PR still keeps the door open on Hive 1.x. All it does is replace some client APIs we were calling that had been deprecated in the 1.x line.

So it makes it a bit easier to upgrade our Hive version at some future point. But there's no hive upgrade in this PR itself.