You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@atlas.apache.org by Sidharth Mishra <si...@gmail.com> on 2021/12/07 20:21:47 UTC

Review Request 73747: ATLAS-4502: Add cassandra and elasticsearch search profile for Atlas

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

Review request for atlas, Ashutosh Mestry, Disha Talreja, Radhika Kundam, and Sarath Subramanian.


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


Repository: atlas


Description
-------

ATLAS-4502: Add cassandra and elasticsearch search profile for Atlas


Diffs
-----

  distro/pom.xml 950a79c85 
  pom.xml 4c92a09ad 
  webapp/pom.xml 858ab9aaf 


Diff: https://reviews.apache.org/r/73747/diff/1/


Testing
-------

Manually tested locally with local Cassandra ans embedded elastic search


Thanks,

Sidharth Mishra

Re: Review Request 73747: ATLAS-4502: Add cassandra and elasticsearch search profile for Atlas

Posted by Nixon Rodrigues <ni...@atlan.com>.

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


Ship it!




Ship It!

- Nixon Rodrigues


On Dec. 9, 2021, 12:26 a.m., Sidharth Mishra wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73747/
> -----------------------------------------------------------
> 
> (Updated Dec. 9, 2021, 12:26 a.m.)
> 
> 
> Review request for atlas, Ashutosh Mestry, Disha Talreja, Radhika Kundam, and Sarath Subramanian.
> 
> 
> Bugs: ATLAS-4502
>     https://issues.apache.org/jira/browse/ATLAS-4502
> 
> 
> Repository: atlas
> 
> 
> Description
> -------
> 
> ATLAS-4502: Add cassandra and elasticsearch search profile for Atlas
> 
> 
> Diffs
> -----
> 
>   distro/pom.xml 950a79c85 
>   pom.xml 4c92a09ad 
>   repository/src/main/java/org/apache/atlas/repository/audit/CassandraBasedAuditRepository.java fb890fc5d 
>   webapp/pom.xml 858ab9aaf 
> 
> 
> Diff: https://reviews.apache.org/r/73747/diff/2/
> 
> 
> Testing
> -------
> 
> Manually tested locally with local Cassandra ans embedded elastic search
> 
> 
> Thanks,
> 
> Sidharth Mishra
> 
>

Re: Review Request 73747: ATLAS-4502: Add cassandra and elasticsearch search profile for Atlas

Posted by Sarath Subramanian <sa...@apache.org>.

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


Ship it!




Ship It!

- Sarath Subramanian


On Dec. 8, 2021, 4:26 p.m., Sidharth Mishra wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73747/
> -----------------------------------------------------------
> 
> (Updated Dec. 8, 2021, 4:26 p.m.)
> 
> 
> Review request for atlas, Ashutosh Mestry, Disha Talreja, Radhika Kundam, and Sarath Subramanian.
> 
> 
> Bugs: ATLAS-4502
>     https://issues.apache.org/jira/browse/ATLAS-4502
> 
> 
> Repository: atlas
> 
> 
> Description
> -------
> 
> ATLAS-4502: Add cassandra and elasticsearch search profile for Atlas
> 
> 
> Diffs
> -----
> 
>   distro/pom.xml 950a79c85 
>   pom.xml 4c92a09ad 
>   repository/src/main/java/org/apache/atlas/repository/audit/CassandraBasedAuditRepository.java fb890fc5d 
>   webapp/pom.xml 858ab9aaf 
> 
> 
> Diff: https://reviews.apache.org/r/73747/diff/2/
> 
> 
> Testing
> -------
> 
> Manually tested locally with local Cassandra ans embedded elastic search
> 
> 
> Thanks,
> 
> Sidharth Mishra
> 
>

Re: Review Request 73747: ATLAS-4502: Add cassandra and elasticsearch search profile for Atlas

Posted by Sidharth Mishra <si...@gmail.com>.

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

(Updated Dec. 9, 2021, 12:26 a.m.)


Review request for atlas, Ashutosh Mestry, Disha Talreja, Radhika Kundam, and Sarath Subramanian.


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


Repository: atlas


Description
-------

ATLAS-4502: Add cassandra and elasticsearch search profile for Atlas


Diffs (updated)
-----

  distro/pom.xml 950a79c85 
  pom.xml 4c92a09ad 
  repository/src/main/java/org/apache/atlas/repository/audit/CassandraBasedAuditRepository.java fb890fc5d 
  webapp/pom.xml 858ab9aaf 


Diff: https://reviews.apache.org/r/73747/diff/2/

Changes: https://reviews.apache.org/r/73747/diff/1-2/


Testing
-------

Manually tested locally with local Cassandra ans embedded elastic search


Thanks,

Sidharth Mishra

Re: Review Request 73747: ATLAS-4502: Add cassandra and elasticsearch search profile for Atlas

Posted by Sarath Subramanian <sa...@apache.org>.

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


Fix it, then Ship it!





pom.xml
Line 586 (original), 586 (patched)
<https://reviews.apache.org/r/73747/#comment312878>

    consider using variable here ${guava.version}


- Sarath Subramanian


On Dec. 7, 2021, 12:21 p.m., Sidharth Mishra wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73747/
> -----------------------------------------------------------
> 
> (Updated Dec. 7, 2021, 12:21 p.m.)
> 
> 
> Review request for atlas, Ashutosh Mestry, Disha Talreja, Radhika Kundam, and Sarath Subramanian.
> 
> 
> Bugs: ATLAS-4502
>     https://issues.apache.org/jira/browse/ATLAS-4502
> 
> 
> Repository: atlas
> 
> 
> Description
> -------
> 
> ATLAS-4502: Add cassandra and elasticsearch search profile for Atlas
> 
> 
> Diffs
> -----
> 
>   distro/pom.xml 950a79c85 
>   pom.xml 4c92a09ad 
>   webapp/pom.xml 858ab9aaf 
> 
> 
> Diff: https://reviews.apache.org/r/73747/diff/1/
> 
> 
> Testing
> -------
> 
> Manually tested locally with local Cassandra ans embedded elastic search
> 
> 
> Thanks,
> 
> Sidharth Mishra
> 
>