Wail Alkowaileet has posted comments on this change. Change subject: [ASTERIXDB-2422][STO] Introduce compressed storage ......................................................................
Patch Set 12: (1 comment) https://asterix-gerrit.ics.uci.edu/#/c/2857/12/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/storage/ICompressionManager.java File asterixdb/asterix-common/src/main/java/org/apache/asterix/common/storage/ICompressionManager.java: PS12, Line 48: getDdlOrDefaultCompressionScheme I added this method to ensure that Metadata.Dataset.BlockLevelStorageCompression includes the compression scheme (either the defined scheme in the DDL or the one in the cluster config) during the dataset creation. This is needed to guard against any compression alteration in the cluster configuration. -- To view, visit https://asterix-gerrit.ics.uci.edu/2857 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: Idde6f37c810c30c7f1a5ee8bcbc1e3e5f4410031 Gerrit-PatchSet: 12 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Wail Alkowaileet <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Dmitry Lychagin <[email protected]> Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: Wail Alkowaileet <[email protected]> Gerrit-HasComments: Yes
