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]: [ASTERIXDB-1995][STO] Abort write txn when index cannot be f...
Date Tue, 25 Jul 2017 22:06:06 GMT
abdullah alamoudi has posted comments on this change.

Change subject: [ASTERIXDB-1995][STO] Abort write txn when index cannot be flushed

Patch Set 4:


Murtadha, this change needs better tests IMO. Can be done later but we need to cover different
cases when it comes to component states and when is the disk full detected!
File asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/

PS4, Line 226: ("target", "io", "dir")
Can we keep this defined in one place?
File asterixdb/asterix-app/src/test/java/org/apache/asterix/test/storage/

PS4, Line 156: Assert.assertEquals(cause.getErrorCode(), expectedException.getErrorCode());
check both error code and component?
File hyracks-fullstack/hyracks/hyracks-storage-am-lsm-common/src/main/java/org/apache/hyracks/storage/am/lsm/common/impls/

PS4, Line 511: lsmIndex.markAsValid(newComponent);
If I recall correctly, in markAsValid, we wait for the pages to be flushed to disk... but
I am not sure.

For sure, this was one of the comments I received on one of my abandoned changes. worth checking
as it may change the meaning of failed flush?

PS4, Line 677: final Optional<ILSMMemoryComponent> any = lsmIndex.getMemoryComponents().stream()
             :                 .filter(c -> c.getState() == ComponentState.INACTIVE ||
c.getState() == ComponentState.READABLE_WRITABLE
             :                         || c.getState() == ComponentState.READABLE_UNWRITABLE_FLUSHING).findAny();
double check object creation in this since it is called frequently!

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I28592c30c788f4a6f44db8b47a84bc77f6b3f8f3
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <>
Gerrit-Reviewer: Dmitry Lychagin <>
Gerrit-Reviewer: Ian Maxon <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Michael Blow <>
Gerrit-Reviewer: Till Westmann <>
Gerrit-Reviewer: Yingyi Bu <>
Gerrit-Reviewer: abdullah alamoudi <>
Gerrit-HasComments: Yes

View raw message