> On Dec. 8, 2016, 9:34 p.m., Anna Szonyi wrote: > > Hi Attila, > > > > Thanks for finding this! Generally I prefer to leave comments out of code, > > however as this is a very particular use-case and the solution is > > non-trivial, in my opinion it would be useful to add a line or two of > > comments on why/in which case the unescape and escape is necessary. > > > > Thanks, > > /Anna
Hey Anna, Thanks for the review! I've added some clarification comments. Would you please check if it is clearer now on your side? Thanks, Attila - Attila ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/54540/#review158574 ----------------------------------------------------------- On Dec. 8, 2016, 9:53 p.m., Attila Szabo wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/54540/ > ----------------------------------------------------------- > > (Updated Dec. 8, 2016, 9:53 p.m.) > > > Review request for Sqoop and Anna Szonyi. > > > Bugs: SQOOP-3074 > https://issues.apache.org/jira/browse/SQOOP-3074 > > > Repository: sqoop-trunk > > > Description > ------- > > The current implementation of Sqoop will fail to compile the ClassWriter > generated Java class in case of UTF-8 characters are available in the > database but the running host don't have UTF-8 or not supporting that. > Meanwhile we'd like to keep up the support of UTF-8 characters for example in > Avro schemas. > A good example for that TestAvroImport#testNonstandardCharactersInColumnName > fails if executed form a shell/env with non UTF-8 locale. > We have to provide an implementation reflects this. > > > Diffs > ----- > > src/java/org/apache/sqoop/avro/AvroUtil.java ee29f140 > src/java/org/apache/sqoop/orm/AvroSchemaGenerator.java 3c31c43a > src/java/org/apache/sqoop/orm/ClassWriter.java 6f6e66b5 > > Diff: https://reviews.apache.org/r/54540/diff/ > > > Testing > ------- > > ant clean test > > ant clean test -Dthirdparty=true -Dsqoop.thirdparty.lib.dir=/root > -Dsqoop.test.mysql.connectstring.host_url=jdbc:mysql://mysqlhostname/ > -Dsqoop.test.mysql.databasename=sqoop_repo -Dsqoop.test.mysql.password=sqoop > -Dsqoop.test.mysql.username=sqoop > -Dsqoop.test.oracle.connectstring=jdbc:oracle:thin:@//oraclehostname/orcl > -Dsqoop.test.postgresql.connectstring.host_url=jdbc:postgresql://postgreshostname/ > -Dsqoop.test.cubrid.connectstring.host_url=jdbc:cubrid:cubridhotname:33000 > -Dsqoop.test.cubrid.connectstring.username=sqoop > -Dsqoop.test.cubrid.connectstring.database=sqoop > -Dsqoop.test.cubrid.connectstring.password=sqoop > -Dmapred.child.java.opts="-Djava.security.egd=file:/dev/../dev/urandom" > -Dtest.timeout=1000000 > > > Thanks, > > Attila Szabo > >