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-1378 Fix NPE on Feed Connect After Restart
Date Fri, 01 Apr 2016 06:18:27 GMT
abdullah alamoudi has posted comments on this change.

Change subject: ASTERIXDB-1378 Fix NPE on Feed Connect After Restart

Patch Set 1:

File asterix-app/src/main/java/org/apache/asterix/file/

Line 237:         ITypeTraits[] typeTraits = DatasetUtils.computeTupleTypeTraits(dataset,
itemType, metaItemType);
> Can't we move more of the computation into the DatasetUtils?
File asterix-app/src/test/java/org/apache/asterix/test/runtime/

Line 98:     int i = 0;
> This is unused.
Yes. sorry I forgot to remove it :)
File asterix-external-data/src/main/java/org/apache/asterix/external/dataflow/

Line 89:             LOGGER.warn("Feed has been interrupted. Closing the feed");
> InterruptedException only happens if somebody interrupts the thread. Why do
I am assuming it is only interrupted by request and so would like to not give a chance to
File asterix-metadata/src/main/java/org/apache/asterix/metadata/utils/

Line 154:             throws AlgebricksException {
> I'm confused about this change. If I reformat the file with the eclipse for
Mmmm, I think this is my problem since I am unsatisfied with out formatter so I added a few
tweaks and I think that I shouldn't have done that :)
File asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/

Line 197:         accessor.prettyPrint();
> Where does this print to? Should this be removed?
Yes. Added it for debugging. will surely remove it

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: If8f8696d252868a8cce0afdbaeda0dd046f99186
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Till Westmann <>
Gerrit-Reviewer: Yingyi Bu <>
Gerrit-Reviewer: abdullah alamoudi <>
Gerrit-HasComments: Yes

View raw message