----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39260/#review104651 -----------------------------------------------------------
src/docs/user/hcatalog.txt (line 120) <https://reviews.apache.org/r/39260/#comment162900> I think we have to fix the text here. Couple of things. This option allows additional partitioning keys to be specified other than the static partitioning keys. These additional partitioning keys will be added to the create table statement to hcatalog as partitioning keys. These key columns should be present in the table otherwise imports will fail. The datatype of these columns in the HCatalog table will be string irrespective of the DB column type. Sqoop has support for dynamic partitions earlier, but was not allowing dynamic partition keys to be provided as part of create hcatalog table src/test/org/apache/sqoop/hcat/HCatalogImportTest.java (line 272) <https://reviews.apache.org/r/39260/#comment162899> Whitespace - Please remove tabs - Venkat Ranganathan On Oct. 26, 2015, 11:33 p.m., Shashank Tandon wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39260/ > ----------------------------------------------------------- > > (Updated Oct. 26, 2015, 11:33 p.m.) > > > Review request for Sqoop and Venkat Ranganathan. > > > Repository: sqoop-trunk > > > Description > ------- > > New Review request for support of Dynamic partitons while importing data. > This is in addition to the old review https://reviews.apache.org/r/38400/ > already exists, but in the review board submitter cannot be changed so adding > another review request to add a new patch for the code review changes. > Functioanlity : Dynamic partition already exists in Sqoop in case of Hive > partitioned table already exists.The new changes will allow to support > dynamic partition if Hive partitioned table does not exists. > This has the list of column to be added as partitioning keys to the created > table along with the static partitioning keys and values provided. For > dynamic partitions keys, the data type of the partition key will be forced to > be string irrespective of the SQL datatype > Also added support for date formats if user wants to do partitioning based on > a date column. > > > Diffs > ----- > > src/docs/user/hcatalog.txt 99ae4f5 > src/java/org/apache/sqoop/SqoopOptions.java db92b30 > src/java/org/apache/sqoop/mapreduce/hcat/SqoopHCatImportHelper.java aba2458 > src/java/org/apache/sqoop/mapreduce/hcat/SqoopHCatUtilities.java 9d5a9ee > src/java/org/apache/sqoop/tool/BaseSqoopTool.java a7f6aec > src/java/org/apache/sqoop/tool/ImportTool.java 39af42c > src/test/org/apache/sqoop/hcat/HCatalogImportTest.java 8db06d1 > > Diff: https://reviews.apache.org/r/39260/diff/ > > > Testing > ------- > > Yes > > > Thanks, > > Shashank Tandon > >
