Murtadha Hubail has submitted this change and it was merged. Change subject: [ASTERIXDB-2503][MD] Consider Datasets Alternating ID ......................................................................
[ASTERIXDB-2503][MD] Consider Datasets Alternating ID - user model changes: no - storage format changes: no - interface changes: no Details: - When initializing the DatasetIdFactory, consider the alternating id of datasets to avoid reusing ids. Change-Id: I2a3ee8f3b998c92afef5e793c7b697682fbf1f17 Reviewed-on: https://asterix-gerrit.ics.uci.edu/3099 Sonar-Qube: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Integration-Tests: Jenkins <[email protected]> Reviewed-by: Till Westmann <[email protected]> --- M asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/MetadataNode.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Anon. E. Moose #1000171: Till Westmann: Looks good to me, approved Jenkins: Verified; No violations found; Verified diff --git a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/MetadataNode.java b/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/MetadataNode.java index abea47f..8cecb9e 100644 --- a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/MetadataNode.java +++ b/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/MetadataNode.java @@ -1371,7 +1371,8 @@ rangeCursor.next(); final ITupleReference ref = rangeCursor.getTuple(); final Dataset ds = valueExtractor.getValue(txnId, ref); - int datasetId = ds.getDatasetId(); + int datasetId = Math.max(ds.getDatasetId(), + DatasetIdFactory.generateAlternatingDatasetId(ds.getDatasetId())); if (mostRecentDatasetId < datasetId) { mostRecentDatasetId = datasetId; } -- To view, visit https://asterix-gerrit.ics.uci.edu/3099 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2a3ee8f3b998c92afef5e793c7b697682fbf1f17 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: stabilization-f69489 Gerrit-Owner: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]>
