You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by Dian Fu <di...@gmail.com> on 2015/09/10 02:50:52 UTC

Review Request 38242: Sqoop2: Provide DateTime type for configuration objects

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

Review request for Sqoop.


Bugs: SQOOP-2559
    https://issues.apache.org/jira/browse/SQOOP-2559


Repository: sqoop-sqoop2


Description
-------

Provide DateTime type for configuration objects per discussion in SQOOP-2554.


Diffs
-----

  common/pom.xml ecd42bc 
  common/src/main/java/org/apache/sqoop/json/util/ConfigInputSerialization.java e2e5b67 
  common/src/main/java/org/apache/sqoop/model/ConfigUtils.java 5d53437 
  common/src/main/java/org/apache/sqoop/model/MConfig.java 6f11d92 
  common/src/main/java/org/apache/sqoop/model/MConfigList.java 2a3c3e4 
  common/src/main/java/org/apache/sqoop/model/MDateTimeInput.java PRE-CREATION 
  common/src/main/java/org/apache/sqoop/model/MInputType.java 0f1fe0a 
  common/src/test/java/org/apache/sqoop/json/util/TestConfigSerialization.java 2a2cfe8 
  common/src/test/java/org/apache/sqoop/model/TestMConfig.java 2cb3989 
  common/src/test/java/org/apache/sqoop/model/TestMConfigList.java fd100b1 
  common/src/test/java/org/apache/sqoop/model/TestMDateTimeInput.java PRE-CREATION 
  connector/connector-sdk/pom.xml 5e20713 
  repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java 6d303c8 
  shell/src/main/java/org/apache/sqoop/shell/utils/ConfigDisplayer.java ab442fe 
  shell/src/main/java/org/apache/sqoop/shell/utils/ConfigFiller.java b45d8df 

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


Testing
-------


Thanks,

Dian Fu


Re: Review Request 38242: Sqoop2: Provide DateTime type for configuration objects

Posted by Jarek Cecho <ja...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38242/#review98766
-----------------------------------------------------------

Ship it!


Ship It!

- Jarek Cecho


On Sept. 10, 2015, 8:33 a.m., Dian Fu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38242/
> -----------------------------------------------------------
> 
> (Updated Sept. 10, 2015, 8:33 a.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-2559
>     https://issues.apache.org/jira/browse/SQOOP-2559
> 
> 
> Repository: sqoop-sqoop2
> 
> 
> Description
> -------
> 
> Provide DateTime type for configuration objects per discussion in SQOOP-2554.
> 
> 
> Diffs
> -----
> 
>   common/pom.xml ecd42bc 
>   common/src/main/java/org/apache/sqoop/json/util/ConfigInputSerialization.java e2e5b67 
>   common/src/main/java/org/apache/sqoop/model/ConfigUtils.java 5d53437 
>   common/src/main/java/org/apache/sqoop/model/MConfig.java 6f11d92 
>   common/src/main/java/org/apache/sqoop/model/MConfigList.java 2a3c3e4 
>   common/src/main/java/org/apache/sqoop/model/MDateTimeInput.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/model/MInputType.java 0f1fe0a 
>   common/src/test/java/org/apache/sqoop/json/util/TestConfigSerialization.java 2a2cfe8 
>   common/src/test/java/org/apache/sqoop/model/TestMConfig.java 2cb3989 
>   common/src/test/java/org/apache/sqoop/model/TestMConfigList.java fd100b1 
>   common/src/test/java/org/apache/sqoop/model/TestMDateTimeInput.java PRE-CREATION 
>   connector/connector-sdk/pom.xml c888aa0 
>   repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java 6d303c8 
>   shell/src/main/java/org/apache/sqoop/shell/utils/ConfigDisplayer.java ab442fe 
>   shell/src/main/java/org/apache/sqoop/shell/utils/ConfigFiller.java b45d8df 
> 
> Diff: https://reviews.apache.org/r/38242/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dian Fu
> 
>


Re: Review Request 38242: Sqoop2: Provide DateTime type for configuration objects

Posted by Dian Fu <di...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38242/
-----------------------------------------------------------

(Updated Sept. 10, 2015, 8:33 a.m.)


Review request for Sqoop.


Bugs: SQOOP-2559
    https://issues.apache.org/jira/browse/SQOOP-2559


Repository: sqoop-sqoop2


Description
-------

Provide DateTime type for configuration objects per discussion in SQOOP-2554.


Diffs (updated)
-----

  common/pom.xml ecd42bc 
  common/src/main/java/org/apache/sqoop/json/util/ConfigInputSerialization.java e2e5b67 
  common/src/main/java/org/apache/sqoop/model/ConfigUtils.java 5d53437 
  common/src/main/java/org/apache/sqoop/model/MConfig.java 6f11d92 
  common/src/main/java/org/apache/sqoop/model/MConfigList.java 2a3c3e4 
  common/src/main/java/org/apache/sqoop/model/MDateTimeInput.java PRE-CREATION 
  common/src/main/java/org/apache/sqoop/model/MInputType.java 0f1fe0a 
  common/src/test/java/org/apache/sqoop/json/util/TestConfigSerialization.java 2a2cfe8 
  common/src/test/java/org/apache/sqoop/model/TestMConfig.java 2cb3989 
  common/src/test/java/org/apache/sqoop/model/TestMConfigList.java fd100b1 
  common/src/test/java/org/apache/sqoop/model/TestMDateTimeInput.java PRE-CREATION 
  connector/connector-sdk/pom.xml c888aa0 
  repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java 6d303c8 
  shell/src/main/java/org/apache/sqoop/shell/utils/ConfigDisplayer.java ab442fe 
  shell/src/main/java/org/apache/sqoop/shell/utils/ConfigFiller.java b45d8df 

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


Testing
-------


Thanks,

Dian Fu


Re: Review Request 38242: Sqoop2: Provide DateTime type for configuration objects

Posted by Dian Fu <di...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38242/
-----------------------------------------------------------

(Updated Sept. 10, 2015, 2:14 a.m.)


Review request for Sqoop.


Bugs: SQOOP-2559
    https://issues.apache.org/jira/browse/SQOOP-2559


Repository: sqoop-sqoop2


Description
-------

Provide DateTime type for configuration objects per discussion in SQOOP-2554.


Diffs (updated)
-----

  common/pom.xml ecd42bc 
  common/src/main/java/org/apache/sqoop/json/util/ConfigInputSerialization.java e2e5b67 
  common/src/main/java/org/apache/sqoop/model/ConfigUtils.java 5d53437 
  common/src/main/java/org/apache/sqoop/model/MConfig.java 6f11d92 
  common/src/main/java/org/apache/sqoop/model/MConfigList.java 2a3c3e4 
  common/src/main/java/org/apache/sqoop/model/MDateTimeInput.java PRE-CREATION 
  common/src/main/java/org/apache/sqoop/model/MInputType.java 0f1fe0a 
  common/src/test/java/org/apache/sqoop/json/util/TestConfigSerialization.java 2a2cfe8 
  common/src/test/java/org/apache/sqoop/model/TestMConfig.java 2cb3989 
  common/src/test/java/org/apache/sqoop/model/TestMConfigList.java fd100b1 
  common/src/test/java/org/apache/sqoop/model/TestMDateTimeInput.java PRE-CREATION 
  connector/connector-sdk/pom.xml c888aa0 
  repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java 6d303c8 
  shell/src/main/java/org/apache/sqoop/shell/utils/ConfigDisplayer.java ab442fe 
  shell/src/main/java/org/apache/sqoop/shell/utils/ConfigFiller.java b45d8df 

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


Testing
-------


Thanks,

Dian Fu


Re: Review Request 38242: Sqoop2: Provide DateTime type for configuration objects

Posted by Dian Fu <di...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38242/
-----------------------------------------------------------

(Updated Sept. 10, 2015, 12:56 a.m.)


Review request for Sqoop.


Bugs: SQOOP-2559
    https://issues.apache.org/jira/browse/SQOOP-2559


Repository: sqoop-sqoop2


Description
-------

Provide DateTime type for configuration objects per discussion in SQOOP-2554.


Diffs (updated)
-----

  common/pom.xml ecd42bc 
  common/src/main/java/org/apache/sqoop/json/util/ConfigInputSerialization.java e2e5b67 
  common/src/main/java/org/apache/sqoop/model/ConfigUtils.java 5d53437 
  common/src/main/java/org/apache/sqoop/model/MConfig.java 6f11d92 
  common/src/main/java/org/apache/sqoop/model/MConfigList.java 2a3c3e4 
  common/src/main/java/org/apache/sqoop/model/MDateTimeInput.java PRE-CREATION 
  common/src/main/java/org/apache/sqoop/model/MInputType.java 0f1fe0a 
  common/src/test/java/org/apache/sqoop/json/util/TestConfigSerialization.java 2a2cfe8 
  common/src/test/java/org/apache/sqoop/model/TestMConfig.java 2cb3989 
  common/src/test/java/org/apache/sqoop/model/TestMConfigList.java fd100b1 
  common/src/test/java/org/apache/sqoop/model/TestMDateTimeInput.java PRE-CREATION 
  connector/connector-sdk/pom.xml c888aa0 
  repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java 6d303c8 
  shell/src/main/java/org/apache/sqoop/shell/utils/ConfigDisplayer.java ab442fe 
  shell/src/main/java/org/apache/sqoop/shell/utils/ConfigFiller.java b45d8df 

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


Testing
-------


Thanks,

Dian Fu