asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ildar Absalyamov (Code Review)" <>
Subject Change in asterixdb[master]: [STO][IDX] Eliminated excess antimatter in LSMBTree
Date Fri, 08 Sep 2017 01:35:19 GMT
Ildar Absalyamov has posted comments on this change.

Change subject: [STO][IDX] Eliminated excess antimatter in LSMBTree

Patch Set 15:


Almost all the casts were eliminated by changing methods and classes to use index-specific
generic parameters.
The only exception is the code in, which could not use BTree-specific interfaces
because it's also used to store RTree tuples in RTreeWithAntimatterTuples (see
File hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/

PS10, Line 26: 
> If this is used only for BTree, rename it to reflect this.
File hyracks-fullstack/hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/tuples/

PS10, Line 36: T createTupleReference()
> Change the return type here to BTreeTypeAwareTupleWriter to avoid the downc
Eliminated by refactoring
File hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/api/

PS10, Line 22: interface
> Add javadocs on each method
File hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/api/

PS10, Line 22: interface
> javadocs
File hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree/src/main/java/org/apache/hyracks/storage/am/lsm/btree/impls/

PS10, Line 331:;
              :                 // we can safely throw away updated tuples in secondary BTree
components, because they correspond to
              :                 /
> Can this be done in a better way where we pass false for updateAware in the
File hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree/src/main/java/org/apache/hyracks/storage/am/lsm/btree/tuples/

PS10, Line 61: 
             :     @Override
             :     protected int getNullFlagsBytes(int numFields) {
             :         // numFields + matter/antimatter bit +
> This is repeated in multiple places. It would be nice if it could be moved 

PS10, Line 100: 
> This should be a constant.
File hyracks-fullstack/hyracks/hyracks-storage-am-lsm-btree/src/main/java/org/apache/hyracks/storage/am/lsm/btree/utils/

PS10, Line 67: elds, int[] filter
> Please use consistent naming for this flag.
File hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-btree-test/src/test/java/org/apache/hyracks/storage/am/lsm/btree/util/

PS10, Line 77: needKeyDu
> I don't think adding a parameter called isPrimary in Hyracks storage level 

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I12a67eff8431b52d1f9051b793a5a64b15c009e9
Gerrit-PatchSet: 15
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ildar Absalyamov <>
Gerrit-Reviewer: Ian Maxon <>
Gerrit-Reviewer: Ildar Absalyamov <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Luo Chen <>
Gerrit-Reviewer: Murtadha Hubail <>
Gerrit-Reviewer: Till Westmann <>
Gerrit-Reviewer: abdullah alamoudi <>
Gerrit-HasComments: Yes

View raw message