----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13635/#review26040 -----------------------------------------------------------
src/java/org/apache/sqoop/orm/ClassWriter.java <https://reviews.apache.org/r/13635/#comment50837> Since I think it's impossible to get timezones here I defaulted to this format. I'm considering adding a -0:00 timezone just for really solid iso8601 compliance (particularly for joda DateTime compatibility). Thoughts? - Erik Selin On Sept. 11, 2013, 8:18 p.m., Erik Selin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13635/ > ----------------------------------------------------------- > > (Updated Sept. 11, 2013, 8:18 p.m.) > > > Review request for Sqoop. > > > Bugs: https://issues.apache.org/jira/browse/SQOOP-1176 > > https://issues.apache.org/jira/browse/https://issues.apache.org/jira/browse/SQOOP-1176 > > > Repository: sqoop-trunk > > > Description > ------- > > Adding support for storing imported data as JSON. > > > Diffs > ----- > > ivy.xml d7486e0 > src/java/org/apache/sqoop/SqoopOptions.java 01805f9 > src/java/org/apache/sqoop/orm/ClassWriter.java 0adc718 > src/java/org/apache/sqoop/tool/BaseSqoopTool.java e02a93b > src/test/com/cloudera/sqoop/TestJsonImport.java PRE-CREATION > > Diff: https://reviews.apache.org/r/13635/diff/ > > > Testing > ------- > > > Thanks, > > Erik Selin > >
