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

Posted to dev@atlas.apache.org by Radhika Kundam via Review Board <no...@reviews.apache.org> on 2022/04/27 05:15:18 UTC

Re: Review Request 73731: ATLAS-4495 Glossary section takes longer time to load when Glossaries have large number of Categories and Terms associated with them

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


Ship it!




Ship It!

- Radhika Kundam


On March 7, 2022, 2:56 a.m., Mandar Ambawane wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73731/
> -----------------------------------------------------------
> 
> (Updated March 7, 2022, 2:56 a.m.)
> 
> 
> Review request for atlas, Ashutosh Mestry, Jayendra Parab, Radhika Kundam, and Sarath Subramanian.
> 
> 
> Bugs: ATLAS-4495
>     https://issues.apache.org/jira/browse/ATLAS-4495
> 
> 
> Repository: atlas
> 
> 
> Description
> -------
> 
> It is observed that, due to existing framework Glossary GET api takes longer time.
> 
> This patch avoids the unnecessary db calls and improve the performance when there is large number of data associated with the Glossary (say 10K terms or categories)
> 
> 
> Diffs
> -----
> 
>   repository/src/main/java/org/apache/atlas/glossary/GlossaryService.java f81b53854 
>   repository/src/main/java/org/apache/atlas/repository/ogm/DataAccess.java f902b2afb 
>   repository/src/main/java/org/apache/atlas/repository/ogm/glossary/AbstractGlossaryDTO.java bee88c61f 
>   repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphRetriever.java 9671df994 
> 
> 
> Diff: https://reviews.apache.org/r/73731/diff/4/
> 
> 
> Testing
> -------
> 
> Manual testing is done.
> 
> PreCommit: https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/1009/consoleFull
> 
> PreCommit: https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/1049/consoleFull
> 
> PreCommit: https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/1053/
> 
> 
> Thanks,
> 
> Mandar Ambawane
> 
>