sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Szabolcs Vasas <vasas.szabo...@gmail.com>
Subject Re: Review Request 62028: Remove Sqoop dependency on deprecated HBase APIs
Date Mon, 11 Sep 2017 09:58:19 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62028/
-----------------------------------------------------------

(Updated Sept. 11, 2017, 9:58 a.m.)


Review request for Sqoop.


Bugs: SQOOP-3232
    https://issues.apache.org/jira/browse/SQOOP-3232


Repository: sqoop-trunk


Description
-------

Sqoop currently depends on pre HBase 1.0 APIs that have been deprecated and will be removed
in the HBase 2.0 release.
The task is to remove the dependency on these old APIs to make sure that the upgrade to a
newer HBase version will be easier in the future.


Diffs (updated)
-----

  src/java/org/apache/sqoop/hbase/HBasePutProcessor.java 032fd38ad0ff13372ae70be47e38db8c4ba8ef8f

  src/java/org/apache/sqoop/mapreduce/HBaseBulkImportJob.java 2bbfffe03844517da9d0d7c94380a8fb57c5eb29

  src/java/org/apache/sqoop/mapreduce/HBaseImportJob.java 523d0a7ede70e16b4e80f8349f08c67eba2e4d01

  src/test/com/cloudera/sqoop/hbase/HBaseTestCase.java 8b29b5fdb00e223a4f2af14b8a9cbfd9ba9d7d83

  src/test/org/apache/sqoop/hbase/TestHBasePutProcessor.java PRE-CREATION 


Diff: https://reviews.apache.org/r/62028/diff/2/

Changes: https://reviews.apache.org/r/62028/diff/1-2/


Testing
-------

Added a new test case for the small refactoring I did.
Ran all unit and third party tests successfully.


Thanks,

Szabolcs Vasas


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message