abdullah alamoudi has posted comments on this change. Change subject: ASTERIXDB-1474: Cannot query dataset with big obj ......................................................................
Patch Set 2: (6 comments) Very minor comments https://asterix-gerrit.ics.uci.edu/#/c/920/2/hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/io/IIOManager.java File hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/io/IIOManager.java: Line 46: public long syncWrite(IFileHandle fHandle, long offset, ByteBuffer[] dataArray) throws HyracksDataException; looking at this method definition, no one would guess that the dataArray is always a two parameters (header and data). I think we can do better :) (at least, a comment explaining) Line 50: public long syncRead(IFileHandle fHandle, long offset, ByteBuffer[] dataArray) throws HyracksDataException; Remove unused? or comment future use? https://asterix-gerrit.ics.uci.edu/#/c/920/2/hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/io/IOManager.java File hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/io/IOManager.java: Line 189: @Override remove method? https://asterix-gerrit.ics.uci.edu/#/c/920/2/hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/util/TreeIndexBufferCacheWarmup.java File hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/util/TreeIndexBufferCacheWarmup.java: Line 47: Am I seeing tabs?? https://asterix-gerrit.ics.uci.edu/#/c/920/2/hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/util/TreeIndexStatsGatherer.java File hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/util/TreeIndexStatsGatherer.java: Line 46: ITreeIndexFrame interiorFrame, ITreeIndexMetaDataFrame metaFrame) Huh! how come Sonar doesn't detect those? https://asterix-gerrit.ics.uci.edu/#/c/920/2/hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/ClockPageReplacementStrategy.java File hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/ClockPageReplacementStrategy.java: Line 228: // TODO(mblow): is dropping pages on the floor enough to adhere to memory budget? I believe it is enough. would be nice to find a better source to confirm -- To view, visit https://asterix-gerrit.ics.uci.edu/920 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2f47c551a053e193519a4337c338934f3e171afd Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Michael Blow <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]> Gerrit-HasComments: Yes
