----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38400/#review99160 -----------------------------------------------------------
src/docs/user/hcatalog.txt (line 113) <https://reviews.apache.org/r/38400/#comment156064> I think this can be eloborated a bit. Sqoop already supports dynamic partitions - as you can see from the tests that we have. What is missing today is the ability to identify during table creation that some partitions should be added to the partition by clause alone (without being in the static partition key=value spec list). src/docs/user/hcatalog.txt (line 129) <https://reviews.apache.org/r/38400/#comment156065> I think this is a separate feature by itself that I think should be in a separate JIRA? src/java/org/apache/sqoop/util/DbUtil.java (line 40) <https://reviews.apache.org/r/38400/#comment156066> Not sure why these are needed. Don't we have columninfo queries that support most of the needs including support for column precision? - Venkat Ranganathan On Sept. 15, 2015, 11:02 a.m., Rakesh Sharma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38400/ > ----------------------------------------------------------- > > (Updated Sept. 15, 2015, 11:02 a.m.) > > > Review request for Sqoop and Venkat Ranganathan. > > > Repository: sqoop-trunk > > > Description > ------- > > Review request for support of Dynamic partitons while importing data. Users > can just tell the list of columns based on which partitioning should happen. > Also support for date formats is added 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 ace90fd > src/java/org/apache/sqoop/mapreduce/hcat/SqoopHCatImportHelper.java 878f765 > src/java/org/apache/sqoop/mapreduce/hcat/SqoopHCatUtilities.java 4070c24 > src/java/org/apache/sqoop/metastore/TableMetaData.java PRE-CREATION > src/java/org/apache/sqoop/tool/BaseSqoopTool.java a7f6aec > src/java/org/apache/sqoop/tool/ImportTool.java 39af42c > src/java/org/apache/sqoop/util/DbUtil.java PRE-CREATION > src/test/org/apache/sqoop/hcat/HCatalogImportTest.java d97b870 > > Diff: https://reviews.apache.org/r/38400/diff/ > > > Testing > ------- > > Yes > > > Thanks, > > Rakesh Sharma > >
