----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12218/#review22638 -----------------------------------------------------------
+1 Looks good to me. - Abraham Elmahrek On July 2, 2013, 12:27 a.m., Mengwei Ding wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12218/ > ----------------------------------------------------------- > > (Updated July 2, 2013, 12:27 a.m.) > > > Review request for Sqoop, Abraham Elmahrek, Hari Shreedharan, and Jarek Cecho. > > > Bugs: SQOOP-997 > https://issues.apache.org/jira/browse/SQOOP-997 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > commit f6b3bb1a61c2e9cd3736c5a05882ace390705536 > Author: Mengwei Ding <[email protected]> > Date: Mon Jul 1 17:19:27 2013 -0700 > > SQOOP-997 provide ability to disable the automatic upgrade for connectors > and framework. > > :100644 100644 0540f6b... 12767d3... M > core/src/main/java/org/apache/sqoop/connector/ConnectorManager.java > :100644 100644 838aaaf... f225fcb... M > core/src/main/java/org/apache/sqoop/core/ConfigurationConstants.java > :100644 100644 a81306b... 008817d... M > core/src/main/java/org/apache/sqoop/framework/FrameworkManager.java > :100644 100644 aa1aa8d... 7a6501f... M > core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java > :100644 100644 306bd90... e77eac8... M > core/src/main/java/org/apache/sqoop/repository/Repository.java > :100755 100755 07b148e... 76596b4... M > dist/src/main/server/conf/sqoop.properties > > > Diffs > ----- > > core/src/main/java/org/apache/sqoop/connector/ConnectorManager.java 0540f6b > core/src/main/java/org/apache/sqoop/core/ConfigurationConstants.java > 838aaaf > core/src/main/java/org/apache/sqoop/framework/FrameworkManager.java a81306b > core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java aa1aa8d > core/src/main/java/org/apache/sqoop/repository/Repository.java 306bd90 > dist/src/main/server/conf/sqoop.properties 07b148e > > Diff: https://reviews.apache.org/r/12218/diff/ > > > Testing > ------- > > > Thanks, > > Mengwei Ding > >
