asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "abdullah alamoudi (Code Review)" <>
Subject Change in asterixdb[master]: Cleanup Buffer Cache
Date Thu, 15 Jun 2017 16:55:11 GMT
abdullah alamoudi has posted comments on this change.

Change subject: Cleanup Buffer Cache

Patch Set 2:

File asterixdb/asterix-common/src/main/java/org/apache/asterix/common/context/

PS1, Line 65: synchronized
> is this synchronized needed anymore?
yes because we need register and unregister to be mutually exclusive. and we're updating the
two maps.

PS1, Line 75: synchronized
> is this synchronized needed anymore?
yes because we need register and unregister to be mutually exclusive. and we're updating the
two maps.
File hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/exceptions/

PS1, Line 111: R 
> Unused

PS1, Line 114: ;
> Unused
File hyracks-fullstack/hyracks/hyracks-api/src/main/java/org/apache/hyracks/api/util/

PS1, Line 48: cksDataException.cre
> Error code
File hyracks-fullstack/hyracks/hyracks-api/src/main/resources/errormsg/

PS1, Line 89: element
> elements

PS1, Line 90: is
> was or is not active
File hyracks-fullstack/hyracks/hyracks-control/hyracks-control-nc/src/main/java/org/apache/hyracks/control/nc/io/

PS1, Line 54: 
> I believe this was for debugging. If so, please remove it.
File hyracks-fullstack/hyracks/hyracks-storage-am-common/src/main/java/org/apache/hyracks/storage/am/common/build/

PS1, Line 85: //2. Node leaves
> It would be nice to add a log message here before destroying. I think it wi
File hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/api/

PS1, Line 83: 
> fix docs
File hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/api/

Line 42: 
> Java doc
File hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/impls/

PS1, Line 742: ception();
> Error code or throw UnsupportedOperationException
Done. This should be gone with a small refactoring of the inverted indexes
File hyracks-fullstack/hyracks/hyracks-storage-am-lsm-invertedindex/src/main/java/org/apache/hyracks/storage/am/lsm/invertedindex/ondisk/

PS1, Line 640: 
> error code
File hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/buffercache/

PS1, Line 799: Exception
> put a comment here to why catching Exception is needed
File hyracks-fullstack/hyracks/hyracks-storage-common/src/main/java/org/apache/hyracks/storage/common/file/

PS1, Line 31: Con
> How about changing to ConcurrentHashMap similar to Asterix FileMapManager a

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I15565b07afdc94ac74c608bfe4480fa09dcf8f1c
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Michael Blow <>
Gerrit-Reviewer: Murtadha Hubail <>
Gerrit-Reviewer: abdullah alamoudi <>
Gerrit-HasComments: Yes

View raw message