asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "abdullah alamoudi (Code Review)" <>
Subject Change in hyracks[master]: Add Support for Upsert Operation
Date Sun, 17 Jan 2016 12:22:27 GMT
abdullah alamoudi has posted comments on this change.

Change subject: Add Support for Upsert Operation

Patch Set 6:

File algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/

Line 76:         return true;
> What's the meaning of Map in isMap()?
It means that the input variables == the output variables.
File algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/

Line 93:     }
> Why does sink operator have any variable, now?
I don't know how or why this change was made. I reverted this one.
File algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/visitors/

Line 47:      */
> Can we explain here the definition of the following terms?
File algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/physical/

Line 116:         }
> better throw exception by adding else { } here.
File hyracks/hyracks-storage-am-btree/src/main/java/org/apache/hyracks/storage/am/btree/impls/

Line 141:     }
> Can we just name as getLowKey()?  I know this key is only used for prior to
File hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/api/

Line 37:     public ITupleReference getSearchKey();
> again, can we rename getLowKey()?
File hyracks/hyracks-storage-am-lsm-btree/src/main/java/org/apache/hyracks/storage/am/lsm/btree/impls/

Line 219:     @Override
> Where are the following two methods used?
removed them. I was using them somewhere but with refactoring, I didn't need them anymore.
File hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/api/

Line 44: 
> Where are the following two methods used?
Removed. Good catch :)
File hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/search/

Line 86:         return queryTuple;
> Probably, better be throwing exception here since this method is only legal
If this throws an exception, then the search of the inverted index will always gets an exception
since this method gets called with the beforeOp. Instead, I made it return a null.

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I2705f43b6e6d187ee29b9ba5a7946d422990022a
Gerrit-PatchSet: 6
Gerrit-Project: hyracks
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Taewoo Kim <>
Gerrit-Reviewer: Till Westmann <>
Gerrit-Reviewer: Young-Seok Kim <>
Gerrit-Reviewer: abdullah alamoudi <>
Gerrit-HasComments: Yes

View raw message