sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Masahiro Yamaguchi" <mas...@kcf.biglobe.ne.jp>
Subject Re: Review Request 27268: Staging-table name check don't need using PGBulkloadManager
Date Tue, 16 Dec 2014 06:59:13 GMT

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

(Updated Dec. 16, 2014, 6:59 a.m.)


Review request for Sqoop.


Changes
-------

I delete description about "--clear-staging-table".
And I add description aobut "pgbulkload.clear.staging.table".
Both changes are written in "connecter.txt".


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


Repository: sqoop-trunk


Description
-------

If we use PGBulkloadManager, the number of staging-tables equals number of map tasks.
And staging-table's name is decided as follows.

tmpTableName = table + "_" + context.getTaskAttemptID().toString();

ExportTools check staging-table's name, but this check don't need using PGBulkloadManager.


Diffs
-----

  src/java/org/apache/sqoop/tool/ExportTool.java 15ecddf 
  src/test/com/cloudera/sqoop/manager/PGBulkloadManagerManualTest.java fc5fd6d 

Diff: https://reviews.apache.org/r/27268/diff/


Testing
-------

done PGBulkloadManagerManualTest.


File Attachments (updated)
----------------

SQOOP-1631_3.patch
  https://reviews.apache.org/media/uploaded/files/2014/12/16/a144a739-f65c-4972-a30a-61b5a3a63907__SQOOP-1631_3.patch
SQOOP-1631_4.patch
  https://reviews.apache.org/media/uploaded/files/2014/12/16/8c5a3b75-e870-4d85-b3ae-2416baa953e5__SQOOP-1631_4.patch


Thanks,

Masahiro Yamaguchi


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