Re: Review Request 68870: ATLAS-2786: updated HBase and Solr clients to use all zookeeper entires

2018-09-27 Thread Apoorv Naik

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



Looks like we need to port it over for Janusgraph as well.

- Apoorv Naik


On Sept. 28, 2018, 3 a.m., Madhan Neethiraj wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68870/
> ---
> 
> (Updated Sept. 28, 2018, 3 a.m.)
> 
> 
> Review request for atlas, Apoorv Naik, Ashutosh Mestry, keval bhatt, Nixon 
> Rodrigues, and Sarath Subramanian.
> 
> 
> Bugs: ATLAS-2786
> https://issues.apache.org/jira/browse/ATLAS-2786
> 
> 
> Repository: atlas
> 
> 
> Description
> ---
> 
> Due to incorrect use of Apache commons configiration class, only the first 
> host specified in zookeeper connection sttring was being used to connect to 
> HBase and Solr. This causes failure when the first zookeeper host is not 
> reachable.
> 
> 
> Diffs
> -
> 
>   
> graphdb/titan0/src/main/java/com/thinkaurelius/titan/diskstorage/solr/Solr5Index.java
>  90d24e4d8 
>   
> repository/src/main/java/org/apache/atlas/repository/audit/HBaseBasedAuditRepository.java
>  774934c7f 
> 
> 
> Diff: https://reviews.apache.org/r/68870/diff/1/
> 
> 
> Testing
> ---
> 
> In an environment with 3 zookeepers, verified that Atlas service is not 
> impacted when any one zookeeper instance (including the first one) is not 
> reachable.
> 
> 
> Thanks,
> 
> Madhan Neethiraj
> 
>



Re: Review Request 68870: ATLAS-2786: updated HBase and Solr clients to use all zookeeper entires

2018-09-27 Thread Apoorv Naik

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


Ship it!




Ship It!

- Apoorv Naik


On Sept. 28, 2018, 3 a.m., Madhan Neethiraj wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/68870/
> ---
> 
> (Updated Sept. 28, 2018, 3 a.m.)
> 
> 
> Review request for atlas, Apoorv Naik, Ashutosh Mestry, keval bhatt, Nixon 
> Rodrigues, and Sarath Subramanian.
> 
> 
> Bugs: ATLAS-2786
> https://issues.apache.org/jira/browse/ATLAS-2786
> 
> 
> Repository: atlas
> 
> 
> Description
> ---
> 
> Due to incorrect use of Apache commons configiration class, only the first 
> host specified in zookeeper connection sttring was being used to connect to 
> HBase and Solr. This causes failure when the first zookeeper host is not 
> reachable.
> 
> 
> Diffs
> -
> 
>   
> graphdb/titan0/src/main/java/com/thinkaurelius/titan/diskstorage/solr/Solr5Index.java
>  90d24e4d8 
>   
> repository/src/main/java/org/apache/atlas/repository/audit/HBaseBasedAuditRepository.java
>  774934c7f 
> 
> 
> Diff: https://reviews.apache.org/r/68870/diff/1/
> 
> 
> Testing
> ---
> 
> In an environment with 3 zookeepers, verified that Atlas service is not 
> impacted when any one zookeeper instance (including the first one) is not 
> reachable.
> 
> 
> Thanks,
> 
> Madhan Neethiraj
> 
>