asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Young-Seok Kim (Code Review)" <>
Subject Change in asterixdb[master]: Introducing Data Replication To AsterixDB
Date Sat, 24 Oct 2015 01:25:26 GMT
Young-Seok Kim has posted comments on this change.

Change subject: Introducing Data Replication To AsterixDB

Patch Set 10:


Please address comments.
File asterix-replication/src/main/java/org/apache/asterix/replication/management/

Line 412:             synchronized (GOODBYE_BUFFER) {
Let's create byte buffer for each request instead of going through sync block and doing IO
in it. This seems less harmful based on the fact that sendGoodbye() are used per LSMDiskComponent
replication request and sendAck() per file transmission.

Line 414:                 socketChannel.write(GOODBYE_BUFFER);
make sure that the write() return value matches the size of buffer you requested to write.

Line 421:                 socketChannel.write(ACK_BUFFER);
make sure that the write() return value matches the size of buffer you requested to write.
File asterix-replication/src/main/java/org/apache/asterix/replication/management/

Line 1048:         synchronized (GOODBYE_BUFFER) {
let's revert to generate obejcts
File asterix-replication/src/main/java/org/apache/asterix/replication/recovery/

Line 119:                     //System.out.println("Remote logs found: " + remoteRecoveryLogs.size());
let's remove
File asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/recovery/

Line 142:                 state = SystemState.NEW_UNIVERSE;

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I729fdd1144dbc9ff039b4bc414494860d7553810
Gerrit-PatchSet: 10
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <>
Gerrit-Reviewer: Ian Maxon <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Murtadha Hubail <>
Gerrit-Reviewer: Yingyi Bu <>
Gerrit-Reviewer: Young-Seok Kim <>
Gerrit-Reviewer: abdullah alamoudi <>
Gerrit-HasComments: Yes

View raw message