You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@atlas.apache.org by Pinal Shah <pi...@freestoneinfotech.com> on 2021/10/11 19:20:16 UTC
Re: Review Request 73641: ATLAS-4447: Update Composite indexes status
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73641/
-----------------------------------------------------------
(Updated Oct. 11, 2021, 7:20 p.m.)
Review request for atlas, Ashutosh Mestry, Jayendra Parab, and Sarath Subramanian.
Changes
-------
added configuration for the patch
added commit per index
Bugs: ATLAS-4447
https://issues.apache.org/jira/browse/ATLAS-4447
Repository: atlas
Description
-------
If by any chance, commposite index are not enabled, this patch fixes the composite index from REGISTERED state to ENABLED state
Diffs (updated)
-----
graphdb/api/src/main/java/org/apache/atlas/repository/graphdb/AtlasGraphManagement.java 7e3b2f440
graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/AtlasJanusGraphManagement.java 1cc7f8b55
intg/src/main/java/org/apache/atlas/AtlasConfiguration.java 20f8f7343
repository/src/main/java/org/apache/atlas/repository/patches/AtlasPatchManager.java d30971afc
repository/src/main/java/org/apache/atlas/repository/patches/UpdateCompositeIndexStatusPatch.java PRE-CREATION
Diff: https://reviews.apache.org/r/73641/diff/3/
Changes: https://reviews.apache.org/r/73641/diff/2-3/
Testing
-------
manually tested
Thanks,
Pinal Shah
Re: Review Request 73641: ATLAS-4447: Update Composite indexes status
Posted by Pinal Shah <pi...@freestoneinfotech.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73641/
-----------------------------------------------------------
(Updated Oct. 12, 2021, 8:52 a.m.)
Review request for atlas, Ashutosh Mestry, Jayendra Parab, and Sarath Subramanian.
Changes
-------
rebase
Bugs: ATLAS-4447
https://issues.apache.org/jira/browse/ATLAS-4447
Repository: atlas
Description
-------
If by any chance, commposite index are not enabled, this patch fixes the composite index from REGISTERED state to ENABLED state
Diffs (updated)
-----
graphdb/api/src/main/java/org/apache/atlas/repository/graphdb/AtlasGraphManagement.java 50d17a223
graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/AtlasJanusGraphManagement.java b3eb07142
intg/src/main/java/org/apache/atlas/AtlasConfiguration.java 20f8f7343
repository/src/main/java/org/apache/atlas/repository/patches/AtlasPatchManager.java d30971afc
repository/src/main/java/org/apache/atlas/repository/patches/UpdateCompositeIndexStatusPatch.java PRE-CREATION
Diff: https://reviews.apache.org/r/73641/diff/5/
Changes: https://reviews.apache.org/r/73641/diff/4-5/
Testing
-------
manually tested
Thanks,
Pinal Shah
Re: Review Request 73641: ATLAS-4447: Update Composite indexes status
Posted by Sarath Subramanian <sa...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73641/#review223602
-----------------------------------------------------------
Ship it!
Ship It!
- Sarath Subramanian
On Oct. 11, 2021, 10:30 p.m., Pinal Shah wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73641/
> -----------------------------------------------------------
>
> (Updated Oct. 11, 2021, 10:30 p.m.)
>
>
> Review request for atlas, Ashutosh Mestry, Jayendra Parab, and Sarath Subramanian.
>
>
> Bugs: ATLAS-4447
> https://issues.apache.org/jira/browse/ATLAS-4447
>
>
> Repository: atlas
>
>
> Description
> -------
>
> If by any chance, commposite index are not enabled, this patch fixes the composite index from REGISTERED state to ENABLED state
>
>
> Diffs
> -----
>
> graphdb/api/src/main/java/org/apache/atlas/repository/graphdb/AtlasGraphManagement.java 7e3b2f440
> graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/AtlasJanusGraphManagement.java 1cc7f8b55
> intg/src/main/java/org/apache/atlas/AtlasConfiguration.java 20f8f7343
> repository/src/main/java/org/apache/atlas/repository/patches/AtlasPatchManager.java d30971afc
> repository/src/main/java/org/apache/atlas/repository/patches/UpdateCompositeIndexStatusPatch.java PRE-CREATION
>
>
> Diff: https://reviews.apache.org/r/73641/diff/4/
>
>
> Testing
> -------
>
> manually tested
>
>
> Thanks,
>
> Pinal Shah
>
>
Re: Review Request 73641: ATLAS-4447: Update Composite indexes status
Posted by Pinal Shah <pi...@freestoneinfotech.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73641/
-----------------------------------------------------------
(Updated Oct. 12, 2021, 5:30 a.m.)
Review request for atlas, Ashutosh Mestry, Jayendra Parab, and Sarath Subramanian.
Changes
-------
changed the configuration to set default true
Bugs: ATLAS-4447
https://issues.apache.org/jira/browse/ATLAS-4447
Repository: atlas
Description
-------
If by any chance, commposite index are not enabled, this patch fixes the composite index from REGISTERED state to ENABLED state
Diffs (updated)
-----
graphdb/api/src/main/java/org/apache/atlas/repository/graphdb/AtlasGraphManagement.java 7e3b2f440
graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/AtlasJanusGraphManagement.java 1cc7f8b55
intg/src/main/java/org/apache/atlas/AtlasConfiguration.java 20f8f7343
repository/src/main/java/org/apache/atlas/repository/patches/AtlasPatchManager.java d30971afc
repository/src/main/java/org/apache/atlas/repository/patches/UpdateCompositeIndexStatusPatch.java PRE-CREATION
Diff: https://reviews.apache.org/r/73641/diff/4/
Changes: https://reviews.apache.org/r/73641/diff/3-4/
Testing
-------
manually tested
Thanks,
Pinal Shah