You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@falcon.apache.org by Venkat Ranganathan <n....@live.com> on 2015/11/26 04:25:38 UTC

Re: Review Request 39261: FALCON-1480 : Gather data transfer details of Hive DR

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

Ship it!


Thanks for working on this.  Who is the consumer of this apart from the UI?   Do ywe  have a CLI option to display these?

- Venkat Ranganathan


On Oct. 25, 2015, 10 p.m., Peeyush Bishnoi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39261/
> -----------------------------------------------------------
> 
> (Updated Oct. 25, 2015, 10 p.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1480
>     https://issues.apache.org/jira/browse/FALCON-1480
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> FALCON-1480: Gather data transfer details of Hive DR
> 
> 
> Diffs
> -----
> 
>   addons/hivedr/pom.xml a1be288 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/HiveDRArgs.java 574524d 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/HiveDRTool.java bebdb0b 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/mapreduce/CopyMapper.java 5eb8acb 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/EventUtils.java f8397ff 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/HiveDRUtils.java d9d6ab0 
>   addons/recipes/hive-disaster-recovery/src/main/resources/hive-disaster-recovery-secure-workflow.xml 74902b4 
>   addons/recipes/hive-disaster-recovery/src/main/resources/hive-disaster-recovery-workflow.xml 72d40a3 
>   metrics/src/main/java/org/apache/falcon/job/HiveReplicationCounters.java PRE-CREATION 
>   metrics/src/main/java/org/apache/falcon/job/JobCountersHandler.java e8b68ff 
>   metrics/src/main/java/org/apache/falcon/job/JobType.java 456e57f 
> 
> Diff: https://reviews.apache.org/r/39261/diff/
> 
> 
> Testing
> -------
> 
> Yes. Manual testing done.
> 
> 
> Thanks,
> 
> Peeyush Bishnoi
> 
>


Re: Review Request 39261: FALCON-1480 : Gather data transfer details of Hive DR

Posted by Peeyush Bishnoi <bp...@yahoo.co.in>.

> On Nov. 26, 2015, 3:25 a.m., Venkat Ranganathan wrote:
> > Thanks for working on this.  Who is the consumer of this apart from the UI?   Do ywe  have a CLI option to display these?

Thank for the review. Currently as per the requirement, UI is only the consumer for captured metrics. But we can have CLI as another option to display the captured replication metrics from graphdb. I will create a separate issue for this to discuss more.


- Peeyush


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


On Oct. 26, 2015, 5 a.m., Peeyush Bishnoi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39261/
> -----------------------------------------------------------
> 
> (Updated Oct. 26, 2015, 5 a.m.)
> 
> 
> Review request for Falcon.
> 
> 
> Bugs: FALCON-1480
>     https://issues.apache.org/jira/browse/FALCON-1480
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> FALCON-1480: Gather data transfer details of Hive DR
> 
> 
> Diffs
> -----
> 
>   addons/hivedr/pom.xml a1be288 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/HiveDRArgs.java 574524d 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/HiveDRTool.java bebdb0b 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/mapreduce/CopyMapper.java 5eb8acb 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/EventUtils.java f8397ff 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/HiveDRUtils.java d9d6ab0 
>   addons/recipes/hive-disaster-recovery/src/main/resources/hive-disaster-recovery-secure-workflow.xml 74902b4 
>   addons/recipes/hive-disaster-recovery/src/main/resources/hive-disaster-recovery-workflow.xml 72d40a3 
>   metrics/src/main/java/org/apache/falcon/job/HiveReplicationCounters.java PRE-CREATION 
>   metrics/src/main/java/org/apache/falcon/job/JobCountersHandler.java e8b68ff 
>   metrics/src/main/java/org/apache/falcon/job/JobType.java 456e57f 
> 
> Diff: https://reviews.apache.org/r/39261/diff/
> 
> 
> Testing
> -------
> 
> Yes. Manual testing done.
> 
> 
> Thanks,
> 
> Peeyush Bishnoi
> 
>