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:

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

Review request for Sqoop.

Bugs: SQOOP-3232

Repository: sqoop-trunk


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/


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


Szabolcs Vasas

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