You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@falcon.apache.org by Paul Isaychuk <pi...@hortonworks.com> on 2014/11/19 19:41:09 UTC

Review Request 28238: [FALCON-906] Add license header where it is needed, remove deprecated inmobi references

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

Review request for Falcon.


Bugs: FALCON-906
    https://issues.apache.org/jira/browse/FALCON-906


Repository: falcon-git


Description
-------

Clean up stuff related to https://issues.apache.org/jira/browse/FALCON-900


Diffs
-----

  falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/PrismProcessDeleteTest.java 1bc61b3 
  falcon-regression/merlin/src/test/java/testng.xml 0e442c0 
  falcon-regression/merlin/src/test/resources/AvailabilityBundle/valid/bundle1/cluster-0.1.xml 3546de6 
  falcon-regression/merlin/src/test/resources/ELbundle/cluster-0.1.xml cdb14ae 
  falcon-regression/merlin/src/test/resources/EmptyInputTagProcess/cluster-0.1.xml cdb14ae 
  falcon-regression/merlin/src/test/resources/EmptyOutputTagProcess/cluster-0.1.xml cdb14ae 
  falcon-regression/merlin/src/test/resources/FeedReplicaltionBundles/cluster-0.1.xml 7d1b089 
  falcon-regression/merlin/src/test/resources/LateDataBundles/valid1/bundle1/cluster-0.1.xml 6a00752 
  falcon-regression/merlin/src/test/resources/OozieExampleInputData/normalInput/log_01.txt 79b343f 
  falcon-regression/merlin/src/test/resources/ReplicationResources/cluster-0.1.xml cdb14ae 
  falcon-regression/merlin/src/test/resources/ReplicationResources/feed-s4Replication.xml 96fa910 
  falcon-regression/merlin/src/test/resources/RetentionBundles/valid/bundle1/cluster-0.1.xml 6a00752 
  falcon-regression/merlin/src/test/resources/RetryTests/valid1/bundle1/cluster-0.1.xml 6a00752 
  falcon-regression/merlin/src/test/resources/availabilityFlag.txt 8b13789 
  falcon-regression/merlin/src/test/resources/feed-s4Replication.xml 4e7d4c3 
  falcon-regression/merlin/src/test/resources/hcat/cluster-0.1.xml 5396835 
  falcon-regression/merlin/src/test/resources/hcat/data/data.txt 4321d67 
  falcon-regression/merlin/src/test/resources/hcat_2/cluster-0.1.xml 9be1b2c 
  falcon-regression/merlin/src/test/resources/log_01.txt 6dbf683 
  falcon-regression/merlin/src/test/resources/updateBundle/cluster-0.1.xml cdb14ae 

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


Testing
-------


Thanks,

Paul Isaychuk


Re: Review Request 28238: [FALCON-906] Add license header where it is needed, remove deprecated inmobi references

Posted by Raghav Gautam <ra...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28238/#review62214
-----------------------------------------------------------



falcon-regression/merlin/src/test/resources/availabilityFlag.txt
<https://reviews.apache.org/r/28238/#comment104229>

    This file is used to as oozie availability flag and is supposed to be blank. We can't add license to it. Instead we need to exclude this from rat check.


- Raghav Gautam


On Nov. 19, 2014, 10:41 a.m., Paul Isaychuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28238/
> -----------------------------------------------------------
> 
> (Updated Nov. 19, 2014, 10:41 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-906
>     https://issues.apache.org/jira/browse/FALCON-906
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Clean up stuff related to https://issues.apache.org/jira/browse/FALCON-900
> 
> 
> Diffs
> -----
> 
>   falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/PrismProcessDeleteTest.java 1bc61b3 
>   falcon-regression/merlin/src/test/java/testng.xml 0e442c0 
>   falcon-regression/merlin/src/test/resources/AvailabilityBundle/valid/bundle1/cluster-0.1.xml 3546de6 
>   falcon-regression/merlin/src/test/resources/ELbundle/cluster-0.1.xml cdb14ae 
>   falcon-regression/merlin/src/test/resources/EmptyInputTagProcess/cluster-0.1.xml cdb14ae 
>   falcon-regression/merlin/src/test/resources/EmptyOutputTagProcess/cluster-0.1.xml cdb14ae 
>   falcon-regression/merlin/src/test/resources/FeedReplicaltionBundles/cluster-0.1.xml 7d1b089 
>   falcon-regression/merlin/src/test/resources/LateDataBundles/valid1/bundle1/cluster-0.1.xml 6a00752 
>   falcon-regression/merlin/src/test/resources/OozieExampleInputData/normalInput/log_01.txt 79b343f 
>   falcon-regression/merlin/src/test/resources/ReplicationResources/cluster-0.1.xml cdb14ae 
>   falcon-regression/merlin/src/test/resources/ReplicationResources/feed-s4Replication.xml 96fa910 
>   falcon-regression/merlin/src/test/resources/RetentionBundles/valid/bundle1/cluster-0.1.xml 6a00752 
>   falcon-regression/merlin/src/test/resources/RetryTests/valid1/bundle1/cluster-0.1.xml 6a00752 
>   falcon-regression/merlin/src/test/resources/availabilityFlag.txt 8b13789 
>   falcon-regression/merlin/src/test/resources/feed-s4Replication.xml 4e7d4c3 
>   falcon-regression/merlin/src/test/resources/hcat/cluster-0.1.xml 5396835 
>   falcon-regression/merlin/src/test/resources/hcat/data/data.txt 4321d67 
>   falcon-regression/merlin/src/test/resources/hcat_2/cluster-0.1.xml 9be1b2c 
>   falcon-regression/merlin/src/test/resources/log_01.txt 6dbf683 
>   falcon-regression/merlin/src/test/resources/updateBundle/cluster-0.1.xml cdb14ae 
> 
> Diff: https://reviews.apache.org/r/28238/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Paul Isaychuk
> 
>


Re: Review Request 28238: [FALCON-906] Add license header where it is needed, remove deprecated inmobi references

Posted by Paul Isaychuk <pi...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28238/
-----------------------------------------------------------

(Updated Nov. 19, 2014, 8:24 p.m.)


Review request for Falcon.


Bugs: FALCON-906
    https://issues.apache.org/jira/browse/FALCON-906


Repository: falcon-git


Description
-------

Clean up stuff related to https://issues.apache.org/jira/browse/FALCON-900


Diffs (updated)
-----


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


Testing
-------


Thanks,

Paul Isaychuk


Re: Review Request 28238: [FALCON-906] Add license header where it is needed, remove deprecated inmobi references

Posted by Raghav Gautam <ra...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28238/#review62192
-----------------------------------------------------------



falcon-regression/merlin/src/test/resources/hcat/data/data.txt
<https://reviews.apache.org/r/28238/#comment104204>

    This file is used to load data in hive table. We can't add license to it. Instead we need to exclude this from rat check.


- Raghav Gautam


On Nov. 19, 2014, 10:41 a.m., Paul Isaychuk wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28238/
> -----------------------------------------------------------
> 
> (Updated Nov. 19, 2014, 10:41 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-906
>     https://issues.apache.org/jira/browse/FALCON-906
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Clean up stuff related to https://issues.apache.org/jira/browse/FALCON-900
> 
> 
> Diffs
> -----
> 
>   falcon-regression/merlin/src/test/java/org/apache/falcon/regression/prism/PrismProcessDeleteTest.java 1bc61b3 
>   falcon-regression/merlin/src/test/java/testng.xml 0e442c0 
>   falcon-regression/merlin/src/test/resources/AvailabilityBundle/valid/bundle1/cluster-0.1.xml 3546de6 
>   falcon-regression/merlin/src/test/resources/ELbundle/cluster-0.1.xml cdb14ae 
>   falcon-regression/merlin/src/test/resources/EmptyInputTagProcess/cluster-0.1.xml cdb14ae 
>   falcon-regression/merlin/src/test/resources/EmptyOutputTagProcess/cluster-0.1.xml cdb14ae 
>   falcon-regression/merlin/src/test/resources/FeedReplicaltionBundles/cluster-0.1.xml 7d1b089 
>   falcon-regression/merlin/src/test/resources/LateDataBundles/valid1/bundle1/cluster-0.1.xml 6a00752 
>   falcon-regression/merlin/src/test/resources/OozieExampleInputData/normalInput/log_01.txt 79b343f 
>   falcon-regression/merlin/src/test/resources/ReplicationResources/cluster-0.1.xml cdb14ae 
>   falcon-regression/merlin/src/test/resources/ReplicationResources/feed-s4Replication.xml 96fa910 
>   falcon-regression/merlin/src/test/resources/RetentionBundles/valid/bundle1/cluster-0.1.xml 6a00752 
>   falcon-regression/merlin/src/test/resources/RetryTests/valid1/bundle1/cluster-0.1.xml 6a00752 
>   falcon-regression/merlin/src/test/resources/availabilityFlag.txt 8b13789 
>   falcon-regression/merlin/src/test/resources/feed-s4Replication.xml 4e7d4c3 
>   falcon-regression/merlin/src/test/resources/hcat/cluster-0.1.xml 5396835 
>   falcon-regression/merlin/src/test/resources/hcat/data/data.txt 4321d67 
>   falcon-regression/merlin/src/test/resources/hcat_2/cluster-0.1.xml 9be1b2c 
>   falcon-regression/merlin/src/test/resources/log_01.txt 6dbf683 
>   falcon-regression/merlin/src/test/resources/updateBundle/cluster-0.1.xml cdb14ae 
> 
> Diff: https://reviews.apache.org/r/28238/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Paul Isaychuk
> 
>