You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@atlas.apache.org by Madhan Neethiraj <ma...@apache.org> on 2016/06/26 09:29:51 UTC

Review Request 49236: ATLAS-950: updated Atlas to support kerberos authentication to Solr

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

Review request for atlas, Abhay Kulkarni, Shwetha GS, Selvamohan Neethiraj, Suma Shivaprasad, and Hemanth Yamijala.


Bugs: ATLAS-950
    https://issues.apache.org/jira/browse/ATLAS-950


Repository: atlas


Description
-------

Updated Atlas to use Solr client 5.5.1 version which supports Kerberos authentication to Solr


Diffs
-----

  pom.xml b8ae6f9 
  titan/pom.xml ae3b284 
  titan/src/main/java/com/thinkaurelius/titan/diskstorage/solr/Solr5Index.java f756a4a 

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


Testing
-------

Verified that Atlas text-search works with Solr 1) that requires no authentication and 2) that requires kerberos authentication


Thanks,

Madhan Neethiraj


Re: Review Request 49236: ATLAS-950: updated Atlas to support kerberos authentication to Solr

Posted by Hemanth Yamijala <yh...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49236/#review139523
-----------------------------------------------------------


Ship it!




Verified the following:

API changes: 

* zkStateReader.updateClusterState() - a boolean parameter was dropped. However, the boolean value being used in previous version of API was 'true' - which stood for immediate update. The new API's javadoc says "Forcibly refresh cluster state from ZK." - So they seem to be semantically equivalent.
* State check for replica state 'SYNC' is removed - this again seems an old artifact: https://mail-archives.apache.org/mod_mbox/lucene-solr-user/201410.mbox/%3CCAEvwVFEvCA=5=g7i2HpEtL=51HedoZksKYN+jYierPwWXpZkuA@mail.gmail.com%3E 
* The HTTP client configurator is required for the Kerberos Support I guess.

Functionality:

* Installed Solr 5.5.1 and verified it is working fine (only in non-secure mode) - Madhan has already indicated it works with Kerberos mode.
* Full text search
* Business catalog functionality - mainly because it uses Lucene. I noticed though we are still using the older versions of Lucene and not packaging the dependencies that come with Solr lucene. Maybe we should take this up separately.
* From Solr UI verified documents being indexed are increasing in graph and full text search indexes.

- Hemanth Yamijala


On June 26, 2016, 9:29 a.m., Madhan Neethiraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49236/
> -----------------------------------------------------------
> 
> (Updated June 26, 2016, 9:29 a.m.)
> 
> 
> Review request for atlas, Abhay Kulkarni, Shwetha GS, Selvamohan Neethiraj, Suma Shivaprasad, and Hemanth Yamijala.
> 
> 
> Bugs: ATLAS-950
>     https://issues.apache.org/jira/browse/ATLAS-950
> 
> 
> Repository: atlas
> 
> 
> Description
> -------
> 
> Updated Atlas to use Solr client 5.5.1 version which supports Kerberos authentication to Solr
> 
> 
> Diffs
> -----
> 
>   pom.xml b8ae6f9 
>   titan/pom.xml ae3b284 
>   titan/src/main/java/com/thinkaurelius/titan/diskstorage/solr/Solr5Index.java f756a4a 
> 
> Diff: https://reviews.apache.org/r/49236/diff/
> 
> 
> Testing
> -------
> 
> Verified that Atlas text-search works with Solr 1) that requires no authentication and 2) that requires kerberos authentication
> 
> 
> Thanks,
> 
> Madhan Neethiraj
> 
>