asterixdb-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Till Westmann (Code Review)" <>
Subject Change in asterixdb[master]: [NO ISSUE][ING] Refactor Active Suspend/Resume Logic
Date Fri, 26 Oct 2018 23:05:01 GMT
Till Westmann has posted comments on this change.

Change subject: [NO ISSUE][ING] Refactor Active Suspend/Resume Logic

Patch Set 2:

File asterixdb/asterix-app/src/main/java/org/apache/asterix/app/active/

PS2, Line 258: LOGGER
pull the logging out (as below)?

PS2, Line 261: registeredListeners
inline (as below)?

PS2, Line 289: dataverseName + '.' + entityName
Just a question: Could we just use the toString of the EntityId here?
File asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/

PS1, Line 728: validateIfResourceIsActiveInFeed
> a) I put it back. However, I feel like this should be a runtime data except
a) Metadata operations are always tricky. As they happen during the compilation phase of a
request a CompilationException doesn't seem to be inadequate - at least at the outermost level.
In other cases where we execute something during compilation (e.g. during constant folding)
we'll execute, get a runtime exception ,and have the compiler report is as a compilation exception.

PS1, Line 2966: validateDatasetSta
> Where I come from, callBeforeXHook is called beforeX :)
I understand the original naming scheme, but I think that the current name is easier to grasp
on first contact.

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I38254582e08d97951a949f7327c8c3d7cf2ab51d
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail <>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Murtadha Hubail <>
Gerrit-Reviewer: Till Westmann <>
Gerrit-HasComments: Yes

View raw message