[
https://issues.apache.org/jira/browse/SQOOP-432?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13194221#comment-13194221
] [email protected] commented on SQOOP-432: ----------------------------------------------------- ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3645/ ----------------------------------------------------------- Review request for Bilung Lee and Jarek Cecho. Summary ------- Patch for SQOOP-432 https://issues.apache.org/jira/browse/SQOOP-432 This addresses bug SQOOP-432. https://issues.apache.org/jira/browse/SQOOP-432 Diffs ----- http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/java/org/apache/sqoop/cli/SqoopParser.java 1236344 http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/java/org/apache/sqoop/hive/TableDefWriter.java 1236344 http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/java/org/apache/sqoop/mapreduce/DataDrivenImportJob.java 1236344 http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/java/org/apache/sqoop/mapreduce/db/DBConfiguration.java 1236344 http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/java/org/apache/sqoop/mapreduce/db/DBInputFormat.java 1236344 http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/java/org/apache/sqoop/mapreduce/db/DBOutputFormat.java 1236344 http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/java/org/apache/sqoop/mapreduce/db/DBRecordReader.java 1236344 http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/java/org/apache/sqoop/mapreduce/db/DataDrivenDBRecordReader.java 1236344 http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/java/org/apache/sqoop/mapreduce/db/OracleDBRecordReader.java 1236344 http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/java/org/apache/sqoop/orm/AvroSchemaGenerator.java 1236344 http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/java/org/apache/sqoop/orm/ClassWriter.java 1236344 http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/java/org/apache/sqoop/tool/ImportTool.java 1236344 http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/test/checkstyle.xml 1236344 http://svn.apache.org/repos/asf/incubator/sqoop/trunk/src/test/com/cloudera/sqoop/TestAvroImport.java 1236344 Diff: https://reviews.apache.org/r/3645/diff Testing ------- 'ant checkstyle' returns no errors after the changes in the patch. Thanks, Joseph > Enforce writing space after if statement > ---------------------------------------- > > Key: SQOOP-432 > URL: https://issues.apache.org/jira/browse/SQOOP-432 > Project: Sqoop > Issue Type: Improvement > Affects Versions: 1.3.0, 1.4.0-incubating > Reporter: Jarek Jarcec Cecho > Assignee: Joseph Boyd > Priority: Minor > Attachments: SQOOP-432.patch > > > We're currently not enforcing (=~ "ant checkstyle" is not enforcing) writing > space after if statement even though we're using such code policy in the > code. Would be nice if the policy would be enforced to be consistent across > entire code base. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
