Mailing-List: contact dev-help@falcon.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@falcon.apache.org
Content-Type: multipart/alternative;
 boundary="===============3615705104218973400=="
MIME-Version: 1.0
Subject: Re: Review Request 35833: FALCON-1188: Falcon support for Hive
 Replication
From: "Ajay Yadava" <ajaynsit@gmail.com>
To: "Srikanth Sundarrajan" <sriksun@hotmail.com>
Cc: "Venkat Ranganathan" <n.r.v@live.com>,
 "Falcon" <dev@falcon.incubator.apache.org>,
 "Ajay Yadava" <ajaynsit@gmail.com>
Date: Mon, 03 Aug 2015 05:45:06 -0000
Message-ID: <20150803054506.4206.97889@reviews.apache.org>
Auto-Submitted: auto-generated
Sender: "Ajay Yadava" <noreply@reviews.apache.org>
References: <20150709035542.1573.80323@reviews.apache.org>
In-Reply-To: <20150709035542.1573.80323@reviews.apache.org>
Reply-To: "Ajay Yadava" <ajaynsit@gmail.com>

--===============3615705104218973400==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit


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

Ship it!


Ship It!

- Ajay Yadava


On July 9, 2015, 3:55 a.m., Venkat Ranganathan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/35833/
> -----------------------------------------------------------
> 
> (Updated July 9, 2015, 3:55 a.m.)
> 
> 
> Review request for Falcon and Srikanth Sundarrajan.
> 
> 
> Bugs: FALCON-1188
>     https://issues.apache.org/jira/browse/FALCON-1188
> 
> 
> Repository: falcon-git
> 
> 
> Description
> -------
> 
> Falcon support for HiveDR
> 
> As referenced in the document attached to FALCON-1188, this patch introduces Falcon support for HiveDR
> 
> It is implemented as recipe and added to Falcon addons package and can be included into the base product using the maven profile hivedr.
> 
> This supports HiveDR in both secure and unsecure clusters.
> 
> This is a large code base and we have repackage it to be in the addons so that we can quickly integrate with falcon master branch and continue further refinements/development in the master branch instead of creating a separate branch introducing merge issues
> 
> This work is the result of contribution of Balu Vellanki, Peeyush Bishnoi, Sowmya Ramesh and Venkat Ranganathan with  help from Hive folks
> 
> 
> Diffs
> -----
> 
>   addons/hivedr/README PRE-CREATION 
>   addons/hivedr/pom.xml PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/DefaultPartitioner.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/EventSourcer.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/HiveDRArgs.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/HiveDROptions.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/HiveDRTool.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/LastReplicatedEvents.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/MetaStoreEventSourcer.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/Partitioner.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/ReplicationEventMetadata.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/exception/HiveReplicationException.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/mapreduce/CopyCommitter.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/mapreduce/CopyMapper.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/mapreduce/CopyReducer.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/DBReplicationStatus.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/DRStatusStore.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/DelimiterUtils.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/EventSourcerUtils.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/EventUtils.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/FileUtils.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/HiveDRStatusStore.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/HiveDRUtils.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/HiveMetastoreUtils.java PRE-CREATION 
>   addons/hivedr/src/main/java/org/apache/falcon/hive/util/ReplicationStatus.java PRE-CREATION 
>   addons/hivedr/src/main/resources/log4j.xml PRE-CREATION 
>   addons/hivedr/src/test/java/org/apache/falcon/hive/DBReplicationStatusTest.java PRE-CREATION 
>   addons/hivedr/src/test/java/org/apache/falcon/hive/DRTest.java PRE-CREATION 
>   addons/hivedr/src/test/java/org/apache/falcon/hive/HiveDRStatusStoreTest.java PRE-CREATION 
>   addons/hivedr/src/test/java/org/apache/falcon/hive/ReplicationStatusTest.java PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/35833/diff/
> 
> 
> Testing
> -------
> 
> HiveDR has been validated both in secure and unsecure cluster configurations.  Checkstyle and RAT checks pass.   There is an associated UI enhancement in Falcon UI that is provided independently.
> Please see the README on component version requirements.
> 
> 
> Thanks,
> 
> Venkat Ranganathan
> 
>


--===============3615705104218973400==--