You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@oozie.apache.org by Mona Chitnis <mo...@yahoo.in> on 2013/01/17 03:25:22 UTC

Review Request: OOZIE-1158 Add hcat sharelib as sub-module

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

Review request for oozie.


Description
-------

Changed POMs to have a HCatalog sharelib sub-module. Exclusions for jackson-mapper-asl and jackson-core-asl are required for working with the correct version - 1.8.8 required by the JSONParser used by HCatalog JMS code.


This addresses bug OOZIE-1158.
    https://issues.apache.org/jira/browse/OOZIE-1158


Diffs
-----

  branches/hcat-intre/core/pom.xml 1434525 
  branches/hcat-intre/hadooplibs/hadoop-test-2/pom.xml 1434525 
  branches/hcat-intre/pom.xml 1434525 
  branches/hcat-intre/sharelib/hcatalog/pom.xml PRE-CREATION 
  branches/hcat-intre/sharelib/hive/pom.xml 1434525 
  branches/hcat-intre/sharelib/pig/pom.xml 1434525 
  branches/hcat-intre/sharelib/pom.xml 1434525 

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


Testing
-------

build is successful. e-2-e testing to follow


Thanks,

Mona Chitnis


Re: Review Request: OOZIE-1158 Add hcat sharelib as sub-module

Posted by Virag Kothari <vi...@yahoo-inc.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8986/#review15465
-----------------------------------------------------------


The sharelib is for jobs to find libraries in hdfs. As Oozie needs to find hcatalog jars in its classpath, this has to be done differently. (Look at hbaselibs and hadooplibs)

- Virag Kothari


On Jan. 17, 2013, 2:25 a.m., Mona Chitnis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8986/
> -----------------------------------------------------------
> 
> (Updated Jan. 17, 2013, 2:25 a.m.)
> 
> 
> Review request for oozie.
> 
> 
> Description
> -------
> 
> Changed POMs to have a HCatalog sharelib sub-module. Exclusions for jackson-mapper-asl and jackson-core-asl are required for working with the correct version - 1.8.8 required by the JSONParser used by HCatalog JMS code.
> 
> 
> This addresses bug OOZIE-1158.
>     https://issues.apache.org/jira/browse/OOZIE-1158
> 
> 
> Diffs
> -----
> 
>   branches/hcat-intre/core/pom.xml 1434525 
>   branches/hcat-intre/hadooplibs/hadoop-test-2/pom.xml 1434525 
>   branches/hcat-intre/pom.xml 1434525 
>   branches/hcat-intre/sharelib/hcatalog/pom.xml PRE-CREATION 
>   branches/hcat-intre/sharelib/hive/pom.xml 1434525 
>   branches/hcat-intre/sharelib/pig/pom.xml 1434525 
>   branches/hcat-intre/sharelib/pom.xml 1434525 
> 
> Diff: https://reviews.apache.org/r/8986/diff/
> 
> 
> Testing
> -------
> 
> build is successful. e-2-e testing to follow
> 
> 
> Thanks,
> 
> Mona Chitnis
> 
>


Re: Review Request: OOZIE-1158 Add hcat sharelib as sub-module

Posted by Virag Kothari <vi...@yahoo-inc.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8986/#review15497
-----------------------------------------------------------


This diff also has sharelib related changes. Can you remove them?

- Virag Kothari


On Jan. 18, 2013, 1:35 a.m., Mona Chitnis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8986/
> -----------------------------------------------------------
> 
> (Updated Jan. 18, 2013, 1:35 a.m.)
> 
> 
> Review request for oozie.
> 
> 
> Description
> -------
> 
> Changed POMs to have a HCatalogLibs sub-module. Exclusions for jackson-mapper-asl and jackson-core-asl in pig and hadoop dependency are required for working with the correct version - 1.8.8 required by the JSONParser used by HCatalog JMS code.
> 
> 
> This addresses bug OOZIE-1158.
>     https://issues.apache.org/jira/browse/OOZIE-1158
> 
> 
> Diffs
> -----
> 
>   branches/hcat-intre/core/pom.xml 1434525 
>   branches/hcat-intre/hadooplibs/hadoop-test-2/pom.xml 1434525 
>   branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml PRE-CREATION 
>   branches/hcat-intre/hcataloglibs/pom.xml PRE-CREATION 
>   branches/hcat-intre/pom.xml 1434525 
>   branches/hcat-intre/resourcelib/README 1434525 
>   branches/hcat-intre/resourcelib/hcatalog-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/resourcelib/webhcat-java-client-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/sharelib/hive/pom.xml 1434525 
>   branches/hcat-intre/sharelib/pig/pom.xml 1434525 
>   branches/hcat-intre/src/main/assemblies/hcataloglib.xml PRE-CREATION 
>   branches/hcat-intre/src/main/assemblies/hcataloglibs.xml PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/8986/diff/
> 
> 
> Testing
> -------
> 
> build and examples successful
> 
> 
> Thanks,
> 
> Mona Chitnis
> 
>


Re: Review Request: OOZIE-1158 Add hcat sharelib as sub-module

Posted by Mona Chitnis <mo...@yahoo.in>.

> On Jan. 18, 2013, 9:41 p.m., Rohini Palaniswamy wrote:
> > branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml, lines 115-119
> > <https://reviews.apache.org/r/8986/diff/3/?file=249980#file249980line115>
> >
> >     This dependency can be removed

yes was waiting for your rework JIRA to go in.


> On Jan. 18, 2013, 9:41 p.m., Rohini Palaniswamy wrote:
> > branches/hcat-intre/sharelib/hive/pom.xml, lines 109-110
> > <https://reviews.apache.org/r/8986/diff/3/?file=249986#file249986line109>
> >
> >     Changes to this file has to be reverted

These sharelib POM changes are part of my own patch before https://issues.apache.org/jira/browse/OOZIE-1123. Now its a correction that these JARs are not required for the jobs themselves, but only for Oozie classpath. Hence these removals to keep the sharelib-hive POM in sync with trunk


- Mona


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


On Jan. 18, 2013, 1:35 a.m., Mona Chitnis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8986/
> -----------------------------------------------------------
> 
> (Updated Jan. 18, 2013, 1:35 a.m.)
> 
> 
> Review request for oozie.
> 
> 
> Description
> -------
> 
> Changed POMs to have a HCatalogLibs sub-module. Exclusions for jackson-mapper-asl and jackson-core-asl in pig and hadoop dependency are required for working with the correct version - 1.8.8 required by the JSONParser used by HCatalog JMS code.
> 
> 
> This addresses bug OOZIE-1158.
>     https://issues.apache.org/jira/browse/OOZIE-1158
> 
> 
> Diffs
> -----
> 
>   branches/hcat-intre/core/pom.xml 1434525 
>   branches/hcat-intre/hadooplibs/hadoop-test-2/pom.xml 1434525 
>   branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml PRE-CREATION 
>   branches/hcat-intre/hcataloglibs/pom.xml PRE-CREATION 
>   branches/hcat-intre/pom.xml 1434525 
>   branches/hcat-intre/resourcelib/README 1434525 
>   branches/hcat-intre/resourcelib/hcatalog-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/resourcelib/webhcat-java-client-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/sharelib/hive/pom.xml 1434525 
>   branches/hcat-intre/sharelib/pig/pom.xml 1434525 
>   branches/hcat-intre/src/main/assemblies/hcataloglib.xml PRE-CREATION 
>   branches/hcat-intre/src/main/assemblies/hcataloglibs.xml PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/8986/diff/
> 
> 
> Testing
> -------
> 
> build and examples successful
> 
> 
> Thanks,
> 
> Mona Chitnis
> 
>


Re: Review Request: OOZIE-1158 Add hcat sharelib as sub-module

Posted by Rohini Palaniswamy <ro...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8986/#review15493
-----------------------------------------------------------



branches/hcat-intre/core/pom.xml
<https://reviews.apache.org/r/8986/#comment33465>

    This should still be here and not part of hcatolglibs as JMSAccessorService is not tied to hcat



branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml
<https://reviews.apache.org/r/8986/#comment33466>

    Can we also explicitly specify hcatalog-core?



branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml
<https://reviews.apache.org/r/8986/#comment33455>

    This dependency can be removed



branches/hcat-intre/sharelib/hive/pom.xml
<https://reviews.apache.org/r/8986/#comment33467>

    Changes to this file has to be reverted


- Rohini Palaniswamy


On Jan. 18, 2013, 1:35 a.m., Mona Chitnis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8986/
> -----------------------------------------------------------
> 
> (Updated Jan. 18, 2013, 1:35 a.m.)
> 
> 
> Review request for oozie.
> 
> 
> Description
> -------
> 
> Changed POMs to have a HCatalogLibs sub-module. Exclusions for jackson-mapper-asl and jackson-core-asl in pig and hadoop dependency are required for working with the correct version - 1.8.8 required by the JSONParser used by HCatalog JMS code.
> 
> 
> This addresses bug OOZIE-1158.
>     https://issues.apache.org/jira/browse/OOZIE-1158
> 
> 
> Diffs
> -----
> 
>   branches/hcat-intre/core/pom.xml 1434525 
>   branches/hcat-intre/hadooplibs/hadoop-test-2/pom.xml 1434525 
>   branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml PRE-CREATION 
>   branches/hcat-intre/hcataloglibs/pom.xml PRE-CREATION 
>   branches/hcat-intre/pom.xml 1434525 
>   branches/hcat-intre/resourcelib/README 1434525 
>   branches/hcat-intre/resourcelib/hcatalog-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/resourcelib/webhcat-java-client-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/sharelib/hive/pom.xml 1434525 
>   branches/hcat-intre/sharelib/pig/pom.xml 1434525 
>   branches/hcat-intre/src/main/assemblies/hcataloglib.xml PRE-CREATION 
>   branches/hcat-intre/src/main/assemblies/hcataloglibs.xml PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/8986/diff/
> 
> 
> Testing
> -------
> 
> build and examples successful
> 
> 
> Thanks,
> 
> Mona Chitnis
> 
>


Re: Review Request: OOZIE-1158 Add hcat sharelib as sub-module

Posted by Mona Chitnis <mo...@yahoo.in>.

> On Jan. 20, 2013, 10:07 p.m., Rohini Palaniswamy wrote:
> > branches/hcat-intre/sharelib/hive/pom.xml, line 109
> > <https://reviews.apache.org/r/8986/diff/4/?file=250287#file250287line109>
> >
> >     Looks like you missed reverting sharelib/hive/pom.xml in this patch too. Otherwise looks good.

Read earlier comment about the sharelib Pom. Changes are intentional


- Mona


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


On Jan. 18, 2013, 11:51 p.m., Mona Chitnis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8986/
> -----------------------------------------------------------
> 
> (Updated Jan. 18, 2013, 11:51 p.m.)
> 
> 
> Review request for oozie.
> 
> 
> Description
> -------
> 
> Changed POMs to have a HCatalogLibs sub-module. Exclusions for jackson-mapper-asl and jackson-core-asl in pig and hadoop dependency are required for working with the correct version - 1.8.8 required by the JSONParser used by HCatalog JMS code.
> 
> 
> This addresses bug OOZIE-1158.
>     https://issues.apache.org/jira/browse/OOZIE-1158
> 
> 
> Diffs
> -----
> 
>   branches/hcat-intre/core/pom.xml 1435402 
>   branches/hcat-intre/hadooplibs/hadoop-test-2/pom.xml 1435402 
>   branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml PRE-CREATION 
>   branches/hcat-intre/hcataloglibs/pom.xml PRE-CREATION 
>   branches/hcat-intre/pom.xml 1435402 
>   branches/hcat-intre/resourcelib/README 1435402 
>   branches/hcat-intre/resourcelib/hcatalog-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/resourcelib/webhcat-java-client-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/sharelib/hive/pom.xml 1435402 
>   branches/hcat-intre/sharelib/pig/pom.xml 1435402 
>   branches/hcat-intre/src/main/assemblies/hcataloglib.xml PRE-CREATION 
>   branches/hcat-intre/src/main/assemblies/hcataloglibs.xml PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/8986/diff/
> 
> 
> Testing
> -------
> 
> build and examples successful
> 
> 
> Thanks,
> 
> Mona Chitnis
> 
>


Re: Review Request: OOZIE-1158 Add hcat sharelib as sub-module

Posted by Rohini Palaniswamy <ro...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8986/#review15531
-----------------------------------------------------------



branches/hcat-intre/sharelib/hive/pom.xml
<https://reviews.apache.org/r/8986/#comment33532>

    Looks like you missed reverting sharelib/hive/pom.xml in this patch too. Otherwise looks good. 


- Rohini Palaniswamy


On Jan. 18, 2013, 11:51 p.m., Mona Chitnis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8986/
> -----------------------------------------------------------
> 
> (Updated Jan. 18, 2013, 11:51 p.m.)
> 
> 
> Review request for oozie.
> 
> 
> Description
> -------
> 
> Changed POMs to have a HCatalogLibs sub-module. Exclusions for jackson-mapper-asl and jackson-core-asl in pig and hadoop dependency are required for working with the correct version - 1.8.8 required by the JSONParser used by HCatalog JMS code.
> 
> 
> This addresses bug OOZIE-1158.
>     https://issues.apache.org/jira/browse/OOZIE-1158
> 
> 
> Diffs
> -----
> 
>   branches/hcat-intre/core/pom.xml 1435402 
>   branches/hcat-intre/hadooplibs/hadoop-test-2/pom.xml 1435402 
>   branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml PRE-CREATION 
>   branches/hcat-intre/hcataloglibs/pom.xml PRE-CREATION 
>   branches/hcat-intre/pom.xml 1435402 
>   branches/hcat-intre/resourcelib/README 1435402 
>   branches/hcat-intre/resourcelib/hcatalog-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/resourcelib/webhcat-java-client-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/sharelib/hive/pom.xml 1435402 
>   branches/hcat-intre/sharelib/pig/pom.xml 1435402 
>   branches/hcat-intre/src/main/assemblies/hcataloglib.xml PRE-CREATION 
>   branches/hcat-intre/src/main/assemblies/hcataloglibs.xml PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/8986/diff/
> 
> 
> Testing
> -------
> 
> build and examples successful
> 
> 
> Thanks,
> 
> Mona Chitnis
> 
>


Re: Review Request: OOZIE-1158 Add hcat sharelib as sub-module

Posted by Mona Chitnis <mo...@yahoo.in>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8986/
-----------------------------------------------------------

(Updated Jan. 22, 2013, 9 p.m.)


Review request for oozie.


Changes
-------

removed incorrect exclusions.
unit tests pass


Description
-------

Changed POMs to have a HCatalogLibs sub-module. Exclusions for jackson-mapper-asl and jackson-core-asl in pig and hadoop dependency are required for working with the correct version - 1.8.8 required by the JSONParser used by HCatalog JMS code.


This addresses bug OOZIE-1158.
    https://issues.apache.org/jira/browse/OOZIE-1158


Diffs (updated)
-----

  branches/hcat-intre/hadooplibs/hadoop-test-2/pom.xml 1437085 
  branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml 1437085 

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


Testing
-------

build and examples successful


Thanks,

Mona Chitnis


Re: Review Request: OOZIE-1158 Add hcat sharelib as sub-module

Posted by Mona Chitnis <mo...@yahoo.in>.

> On Jan. 21, 2013, 8:23 p.m., Virag Kothari wrote:
> > branches/hcat-intre/sharelib/pig/pom.xml, line 45
> > <https://reviews.apache.org/r/8986/diff/4/?file=250288#file250288line45>
> >
> >     Revert this change in sharelib-pig while committing

will do. 


> On Jan. 21, 2013, 8:23 p.m., Virag Kothari wrote:
> > branches/hcat-intre/sharelib/hive/pom.xml, line 109
> > <https://reviews.apache.org/r/8986/diff/4/?file=250287#file250287line109>
> >
> >     Revert this change while committing

same comment as earlier. These additions were introduced in OOZIE-1123 and were not useful. So removal is intentional


- Mona


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


On Jan. 18, 2013, 11:51 p.m., Mona Chitnis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8986/
> -----------------------------------------------------------
> 
> (Updated Jan. 18, 2013, 11:51 p.m.)
> 
> 
> Review request for oozie.
> 
> 
> Description
> -------
> 
> Changed POMs to have a HCatalogLibs sub-module. Exclusions for jackson-mapper-asl and jackson-core-asl in pig and hadoop dependency are required for working with the correct version - 1.8.8 required by the JSONParser used by HCatalog JMS code.
> 
> 
> This addresses bug OOZIE-1158.
>     https://issues.apache.org/jira/browse/OOZIE-1158
> 
> 
> Diffs
> -----
> 
>   branches/hcat-intre/core/pom.xml 1435402 
>   branches/hcat-intre/hadooplibs/hadoop-test-2/pom.xml 1435402 
>   branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml PRE-CREATION 
>   branches/hcat-intre/hcataloglibs/pom.xml PRE-CREATION 
>   branches/hcat-intre/pom.xml 1435402 
>   branches/hcat-intre/resourcelib/README 1435402 
>   branches/hcat-intre/resourcelib/hcatalog-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/resourcelib/webhcat-java-client-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/sharelib/hive/pom.xml 1435402 
>   branches/hcat-intre/sharelib/pig/pom.xml 1435402 
>   branches/hcat-intre/src/main/assemblies/hcataloglib.xml PRE-CREATION 
>   branches/hcat-intre/src/main/assemblies/hcataloglibs.xml PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/8986/diff/
> 
> 
> Testing
> -------
> 
> build and examples successful
> 
> 
> Thanks,
> 
> Mona Chitnis
> 
>


Re: Review Request: OOZIE-1158 Add hcat sharelib as sub-module

Posted by Virag Kothari <vi...@yahoo-inc.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8986/#review15552
-----------------------------------------------------------


+1


branches/hcat-intre/sharelib/hive/pom.xml
<https://reviews.apache.org/r/8986/#comment33558>

    Revert this change while committing



branches/hcat-intre/sharelib/pig/pom.xml
<https://reviews.apache.org/r/8986/#comment33559>

    Revert this change in sharelib-pig while committing


- Virag Kothari


On Jan. 18, 2013, 11:51 p.m., Mona Chitnis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8986/
> -----------------------------------------------------------
> 
> (Updated Jan. 18, 2013, 11:51 p.m.)
> 
> 
> Review request for oozie.
> 
> 
> Description
> -------
> 
> Changed POMs to have a HCatalogLibs sub-module. Exclusions for jackson-mapper-asl and jackson-core-asl in pig and hadoop dependency are required for working with the correct version - 1.8.8 required by the JSONParser used by HCatalog JMS code.
> 
> 
> This addresses bug OOZIE-1158.
>     https://issues.apache.org/jira/browse/OOZIE-1158
> 
> 
> Diffs
> -----
> 
>   branches/hcat-intre/core/pom.xml 1435402 
>   branches/hcat-intre/hadooplibs/hadoop-test-2/pom.xml 1435402 
>   branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml PRE-CREATION 
>   branches/hcat-intre/hcataloglibs/pom.xml PRE-CREATION 
>   branches/hcat-intre/pom.xml 1435402 
>   branches/hcat-intre/resourcelib/README 1435402 
>   branches/hcat-intre/resourcelib/hcatalog-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/resourcelib/webhcat-java-client-0.4.1.jar UNKNOWN 
>   branches/hcat-intre/sharelib/hive/pom.xml 1435402 
>   branches/hcat-intre/sharelib/pig/pom.xml 1435402 
>   branches/hcat-intre/src/main/assemblies/hcataloglib.xml PRE-CREATION 
>   branches/hcat-intre/src/main/assemblies/hcataloglibs.xml PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/8986/diff/
> 
> 
> Testing
> -------
> 
> build and examples successful
> 
> 
> Thanks,
> 
> Mona Chitnis
> 
>


Re: Review Request: OOZIE-1158 Add hcat sharelib as sub-module

Posted by Mona Chitnis <mo...@yahoo.in>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8986/
-----------------------------------------------------------

(Updated Jan. 18, 2013, 11:51 p.m.)


Review request for oozie.


Changes
-------

Addressed review comments


Description
-------

Changed POMs to have a HCatalogLibs sub-module. Exclusions for jackson-mapper-asl and jackson-core-asl in pig and hadoop dependency are required for working with the correct version - 1.8.8 required by the JSONParser used by HCatalog JMS code.


This addresses bug OOZIE-1158.
    https://issues.apache.org/jira/browse/OOZIE-1158


Diffs (updated)
-----

  branches/hcat-intre/core/pom.xml 1435402 
  branches/hcat-intre/hadooplibs/hadoop-test-2/pom.xml 1435402 
  branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml PRE-CREATION 
  branches/hcat-intre/hcataloglibs/pom.xml PRE-CREATION 
  branches/hcat-intre/pom.xml 1435402 
  branches/hcat-intre/resourcelib/README 1435402 
  branches/hcat-intre/resourcelib/hcatalog-0.4.1.jar UNKNOWN 
  branches/hcat-intre/resourcelib/webhcat-java-client-0.4.1.jar UNKNOWN 
  branches/hcat-intre/sharelib/hive/pom.xml 1435402 
  branches/hcat-intre/sharelib/pig/pom.xml 1435402 
  branches/hcat-intre/src/main/assemblies/hcataloglib.xml PRE-CREATION 
  branches/hcat-intre/src/main/assemblies/hcataloglibs.xml PRE-CREATION 

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


Testing
-------

build and examples successful


Thanks,

Mona Chitnis


Re: Review Request: OOZIE-1158 Add hcat sharelib as sub-module

Posted by Mona Chitnis <mo...@yahoo.in>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8986/
-----------------------------------------------------------

(Updated Jan. 18, 2013, 1:35 a.m.)


Review request for oozie.


Changes
-------

Removed the stop-gap 'resourcelib' dir that was used to check-in temp hcatalog jars


Description
-------

Changed POMs to have a HCatalogLibs sub-module. Exclusions for jackson-mapper-asl and jackson-core-asl in pig and hadoop dependency are required for working with the correct version - 1.8.8 required by the JSONParser used by HCatalog JMS code.


This addresses bug OOZIE-1158.
    https://issues.apache.org/jira/browse/OOZIE-1158


Diffs (updated)
-----

  branches/hcat-intre/core/pom.xml 1434525 
  branches/hcat-intre/hadooplibs/hadoop-test-2/pom.xml 1434525 
  branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml PRE-CREATION 
  branches/hcat-intre/hcataloglibs/pom.xml PRE-CREATION 
  branches/hcat-intre/pom.xml 1434525 
  branches/hcat-intre/resourcelib/README 1434525 
  branches/hcat-intre/resourcelib/hcatalog-0.4.1.jar UNKNOWN 
  branches/hcat-intre/resourcelib/webhcat-java-client-0.4.1.jar UNKNOWN 
  branches/hcat-intre/sharelib/hive/pom.xml 1434525 
  branches/hcat-intre/sharelib/pig/pom.xml 1434525 
  branches/hcat-intre/src/main/assemblies/hcataloglib.xml PRE-CREATION 
  branches/hcat-intre/src/main/assemblies/hcataloglibs.xml PRE-CREATION 

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


Testing
-------

build and examples successful


Thanks,

Mona Chitnis


Re: Review Request: OOZIE-1158 Add hcat sharelib as sub-module

Posted by Mona Chitnis <mo...@yahoo.in>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8986/
-----------------------------------------------------------

(Updated Jan. 18, 2013, 1:32 a.m.)


Review request for oozie.


Changes
-------

Addressed Virag's review comment


Description (updated)
-------

Changed POMs to have a HCatalogLibs sub-module. Exclusions for jackson-mapper-asl and jackson-core-asl in pig and hadoop dependency are required for working with the correct version - 1.8.8 required by the JSONParser used by HCatalog JMS code.


This addresses bug OOZIE-1158.
    https://issues.apache.org/jira/browse/OOZIE-1158


Diffs (updated)
-----

  branches/hcat-intre/core/pom.xml 1434525 
  branches/hcat-intre/hadooplibs/hadoop-test-2/pom.xml 1434525 
  branches/hcat-intre/hcataloglibs/hcatalog-0.6.0-SNAPSHOT/pom.xml PRE-CREATION 
  branches/hcat-intre/hcataloglibs/pom.xml PRE-CREATION 
  branches/hcat-intre/pom.xml 1434525 
  branches/hcat-intre/sharelib/hive/pom.xml 1434525 
  branches/hcat-intre/sharelib/pig/pom.xml 1434525 
  branches/hcat-intre/src/main/assemblies/hcataloglib.xml PRE-CREATION 
  branches/hcat-intre/src/main/assemblies/hcataloglibs.xml PRE-CREATION 

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


Testing (updated)
-------

build and examples successful


Thanks,

Mona Chitnis