>From Ali Alsuliman <[email protected]>: Ali Alsuliman has submitted this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18776 )
Change subject: [ASTERIXDB-3498][COMP] Use sorted cluster locations ...................................................................... [ASTERIXDB-3498][COMP] Use sorted cluster locations - user model changes: no - storage format changes: no - interface changes: no Details: For operators that do not specify the locations where they should be run, use the sorted cluster locations so that they are aligned with the dataset cluster locations which is also sorted. Ext-ref: MB-63356 Change-Id: I98af3329c1701560fd9d12d6587fcf1b3e35a842 Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18776 Tested-by: Ali Alsuliman <[email protected]> Reviewed-by: Ali Alsuliman <[email protected]> Reviewed-by: Hussain Towaileb <[email protected]> --- M asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataProvider.java 1 file changed, 25 insertions(+), 1 deletion(-) Approvals: Ali Alsuliman: Looks good to me, but someone else must approve; Verified Hussain Towaileb: Looks good to me, approved Anon. E. Moose #1000171: diff --git a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataProvider.java b/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataProvider.java index 53cf3d0..46ba9dd 100644 --- a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataProvider.java +++ b/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataProvider.java @@ -960,7 +960,7 @@ } public AlgebricksAbsolutePartitionConstraint getClusterLocations() { - return appCtx.getClusterStateManager().getClusterLocations(); + return appCtx.getClusterStateManager().getSortedClusterLocations(); } public Pair<IOperatorDescriptor, AlgebricksPartitionConstraint> buildExternalDataLookupRuntime( -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18776 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: neo Gerrit-Change-Id: I98af3329c1701560fd9d12d6587fcf1b3e35a842 Gerrit-Change-Number: 18776 Gerrit-PatchSet: 3 Gerrit-Owner: Ali Alsuliman <[email protected]> Gerrit-Reviewer: Ali Alsuliman <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Hussain Towaileb <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Peeyush Gupta <[email protected]> Gerrit-MessageType: merged
