Details

    • Type: Sub-task Sub-task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.4.0-incubating
    • Component/s: None
    • Labels:
      None

      Description

      Refer to SQOOP-371 for details.

        Activity

        Hide
        Hudson added a comment -

        Integrated in Sqoop-jdk-1.6 #51 (See https://builds.apache.org/job/Sqoop-jdk-1.6/51/)
        SQOOP-381: Migrate cli and config packages to new name space

        blee : http://svn.apache.org/viewvc/?view=rev&rev=1195865
        Files :

        • /incubator/sqoop/trunk/src/java/com/cloudera/sqoop/cli/RelatedOptions.java
        • /incubator/sqoop/trunk/src/java/com/cloudera/sqoop/cli/SqoopParser.java
        • /incubator/sqoop/trunk/src/java/com/cloudera/sqoop/cli/ToolOptions.java
        • /incubator/sqoop/trunk/src/java/com/cloudera/sqoop/config/ConfigurationConstants.java
        • /incubator/sqoop/trunk/src/java/com/cloudera/sqoop/config/ConfigurationHelper.java
        • /incubator/sqoop/trunk/src/java/org/apache/sqoop/cli
        • /incubator/sqoop/trunk/src/java/org/apache/sqoop/cli/RelatedOptions.java
        • /incubator/sqoop/trunk/src/java/org/apache/sqoop/cli/SqoopParser.java
        • /incubator/sqoop/trunk/src/java/org/apache/sqoop/cli/ToolOptions.java
        • /incubator/sqoop/trunk/src/java/org/apache/sqoop/config
        • /incubator/sqoop/trunk/src/java/org/apache/sqoop/config/ConfigurationConstants.java
        • /incubator/sqoop/trunk/src/java/org/apache/sqoop/config/ConfigurationHelper.java
        • /incubator/sqoop/trunk/src/test/com/cloudera/sqoop/manager/PostgresqlTest.java
        Show
        Hudson added a comment - Integrated in Sqoop-jdk-1.6 #51 (See https://builds.apache.org/job/Sqoop-jdk-1.6/51/ ) SQOOP-381 : Migrate cli and config packages to new name space blee : http://svn.apache.org/viewvc/?view=rev&rev=1195865 Files : /incubator/sqoop/trunk/src/java/com/cloudera/sqoop/cli/RelatedOptions.java /incubator/sqoop/trunk/src/java/com/cloudera/sqoop/cli/SqoopParser.java /incubator/sqoop/trunk/src/java/com/cloudera/sqoop/cli/ToolOptions.java /incubator/sqoop/trunk/src/java/com/cloudera/sqoop/config/ConfigurationConstants.java /incubator/sqoop/trunk/src/java/com/cloudera/sqoop/config/ConfigurationHelper.java /incubator/sqoop/trunk/src/java/org/apache/sqoop/cli /incubator/sqoop/trunk/src/java/org/apache/sqoop/cli/RelatedOptions.java /incubator/sqoop/trunk/src/java/org/apache/sqoop/cli/SqoopParser.java /incubator/sqoop/trunk/src/java/org/apache/sqoop/cli/ToolOptions.java /incubator/sqoop/trunk/src/java/org/apache/sqoop/config /incubator/sqoop/trunk/src/java/org/apache/sqoop/config/ConfigurationConstants.java /incubator/sqoop/trunk/src/java/org/apache/sqoop/config/ConfigurationHelper.java /incubator/sqoop/trunk/src/test/com/cloudera/sqoop/manager/PostgresqlTest.java
        Hide
        Bilung Lee added a comment -

        Patch committed. Thanks, Jarcec!

        Show
        Bilung Lee added a comment - Patch committed. Thanks, Jarcec!
        Hide
        jiraposter@reviews.apache.org added a comment -

        -----------------------------------------------------------
        This is an automatically generated e-mail. To reply, visit:
        https://reviews.apache.org/r/2613/#review2981
        -----------------------------------------------------------

        Ship it!

        Thanks for help on namespace migration, Jarcec!

        • Bilung

        On 2011-10-29 14:50:14, Jarek Jarcec wrote:

        -----------------------------------------------------------

        This is an automatically generated e-mail. To reply, visit:

        https://reviews.apache.org/r/2613/

        -----------------------------------------------------------

        (Updated 2011-10-29 14:50:14)

        Review request for Sqoop and Bilung Lee.

        Summary

        -------

        I've migrated cli and config packages.

        This addresses bug SQOOP-381.

        https://issues.apache.org/jira/browse/SQOOP-381

        Diffs

        -----

        /src/java/com/cloudera/sqoop/cli/RelatedOptions.java 1194890

        /src/java/com/cloudera/sqoop/cli/SqoopParser.java 1194890

        /src/java/com/cloudera/sqoop/cli/ToolOptions.java 1194890

        /src/java/com/cloudera/sqoop/config/ConfigurationConstants.java 1194890

        /src/java/com/cloudera/sqoop/config/ConfigurationHelper.java 1194890

        /src/java/org/apache/sqoop/cli/RelatedOptions.java PRE-CREATION

        /src/java/org/apache/sqoop/cli/SqoopParser.java PRE-CREATION

        /src/java/org/apache/sqoop/cli/ToolOptions.java PRE-CREATION

        /src/java/org/apache/sqoop/config/ConfigurationConstants.java PRE-CREATION

        /src/java/org/apache/sqoop/config/ConfigurationHelper.java PRE-CREATION

        Diff: https://reviews.apache.org/r/2613/diff

        Testing

        -------

        Thanks,

        Jarek

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2613/#review2981 ----------------------------------------------------------- Ship it! Thanks for help on namespace migration, Jarcec! Bilung On 2011-10-29 14:50:14, Jarek Jarcec wrote: ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2613/ ----------------------------------------------------------- (Updated 2011-10-29 14:50:14) Review request for Sqoop and Bilung Lee. Summary ------- I've migrated cli and config packages. This addresses bug SQOOP-381 . https://issues.apache.org/jira/browse/SQOOP-381 Diffs ----- /src/java/com/cloudera/sqoop/cli/RelatedOptions.java 1194890 /src/java/com/cloudera/sqoop/cli/SqoopParser.java 1194890 /src/java/com/cloudera/sqoop/cli/ToolOptions.java 1194890 /src/java/com/cloudera/sqoop/config/ConfigurationConstants.java 1194890 /src/java/com/cloudera/sqoop/config/ConfigurationHelper.java 1194890 /src/java/org/apache/sqoop/cli/RelatedOptions.java PRE-CREATION /src/java/org/apache/sqoop/cli/SqoopParser.java PRE-CREATION /src/java/org/apache/sqoop/cli/ToolOptions.java PRE-CREATION /src/java/org/apache/sqoop/config/ConfigurationConstants.java PRE-CREATION /src/java/org/apache/sqoop/config/ConfigurationHelper.java PRE-CREATION Diff: https://reviews.apache.org/r/2613/diff Testing ------- Thanks, Jarek
        Hide
        jiraposter@reviews.apache.org added a comment -

        -----------------------------------------------------------
        This is an automatically generated e-mail. To reply, visit:
        https://reviews.apache.org/r/2613/
        -----------------------------------------------------------

        (Updated 2011-10-29 14:50:14.264155)

        Review request for Sqoop and Bilung Lee.

        Changes
        -------

        Forgot to add new files.

        Summary
        -------

        I've migrated cli and config packages.

        This addresses bug SQOOP-381.
        https://issues.apache.org/jira/browse/SQOOP-381

        Diffs (updated)


        /src/java/com/cloudera/sqoop/cli/RelatedOptions.java 1194890
        /src/java/com/cloudera/sqoop/cli/SqoopParser.java 1194890
        /src/java/com/cloudera/sqoop/cli/ToolOptions.java 1194890
        /src/java/com/cloudera/sqoop/config/ConfigurationConstants.java 1194890
        /src/java/com/cloudera/sqoop/config/ConfigurationHelper.java 1194890
        /src/java/org/apache/sqoop/cli/RelatedOptions.java PRE-CREATION
        /src/java/org/apache/sqoop/cli/SqoopParser.java PRE-CREATION
        /src/java/org/apache/sqoop/cli/ToolOptions.java PRE-CREATION
        /src/java/org/apache/sqoop/config/ConfigurationConstants.java PRE-CREATION
        /src/java/org/apache/sqoop/config/ConfigurationHelper.java PRE-CREATION

        Diff: https://reviews.apache.org/r/2613/diff

        Testing
        -------

        Thanks,

        Jarek

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2613/ ----------------------------------------------------------- (Updated 2011-10-29 14:50:14.264155) Review request for Sqoop and Bilung Lee. Changes ------- Forgot to add new files. Summary ------- I've migrated cli and config packages. This addresses bug SQOOP-381 . https://issues.apache.org/jira/browse/SQOOP-381 Diffs (updated) /src/java/com/cloudera/sqoop/cli/RelatedOptions.java 1194890 /src/java/com/cloudera/sqoop/cli/SqoopParser.java 1194890 /src/java/com/cloudera/sqoop/cli/ToolOptions.java 1194890 /src/java/com/cloudera/sqoop/config/ConfigurationConstants.java 1194890 /src/java/com/cloudera/sqoop/config/ConfigurationHelper.java 1194890 /src/java/org/apache/sqoop/cli/RelatedOptions.java PRE-CREATION /src/java/org/apache/sqoop/cli/SqoopParser.java PRE-CREATION /src/java/org/apache/sqoop/cli/ToolOptions.java PRE-CREATION /src/java/org/apache/sqoop/config/ConfigurationConstants.java PRE-CREATION /src/java/org/apache/sqoop/config/ConfigurationHelper.java PRE-CREATION Diff: https://reviews.apache.org/r/2613/diff Testing ------- Thanks, Jarek
        Hide
        jiraposter@reviews.apache.org added a comment -

        -----------------------------------------------------------
        This is an automatically generated e-mail. To reply, visit:
        https://reviews.apache.org/r/2613/
        -----------------------------------------------------------

        Review request for Sqoop and Bilung Lee.

        Summary
        -------

        I've migrated cli and config packages.

        This addresses bug SQOOP-381.
        https://issues.apache.org/jira/browse/SQOOP-381

        Diffs


        /src/java/com/cloudera/sqoop/cli/RelatedOptions.java 1194890
        /src/java/com/cloudera/sqoop/cli/SqoopParser.java 1194890
        /src/java/com/cloudera/sqoop/cli/ToolOptions.java 1194890
        /src/java/com/cloudera/sqoop/config/ConfigurationConstants.java 1194890
        /src/java/com/cloudera/sqoop/config/ConfigurationHelper.java 1194890

        Diff: https://reviews.apache.org/r/2613/diff

        Testing
        -------

        Thanks,

        Jarek

        Show
        jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2613/ ----------------------------------------------------------- Review request for Sqoop and Bilung Lee. Summary ------- I've migrated cli and config packages. This addresses bug SQOOP-381 . https://issues.apache.org/jira/browse/SQOOP-381 Diffs /src/java/com/cloudera/sqoop/cli/RelatedOptions.java 1194890 /src/java/com/cloudera/sqoop/cli/SqoopParser.java 1194890 /src/java/com/cloudera/sqoop/cli/ToolOptions.java 1194890 /src/java/com/cloudera/sqoop/config/ConfigurationConstants.java 1194890 /src/java/com/cloudera/sqoop/config/ConfigurationHelper.java 1194890 Diff: https://reviews.apache.org/r/2613/diff Testing ------- Thanks, Jarek

          People

          • Assignee:
            Jarek Jarcec Cecho
            Reporter:
            Bilung Lee
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development