asterixdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ian Maxon (Code Review)" <>
Subject Change in asterixdb[master]: Changed metadata storage format for nullable field types. Mo...
Date Thu, 16 Jul 2015 23:15:43 GMT
Ian Maxon has posted comments on this change.

Change subject: Changed metadata storage format for nullable field types. Moved field name
generation to the client out of metadata node code. Changed naming scheme for autogenerated

Patch Set 1:


Just a few comments. Very cool otherwise! Much cleaner than the weird on the fly construction
of UNION(NULL, fooType) wherever one needed to decide if something was optional or not.
File asterix-metadata/src/main/java/edu/uci/ics/asterix/metadata/bootstrap/

Line 120:         //                FEED_ACTIVITY_DATASET_ID, true, new int[] { 0, 1, 2, 3
Is this block supposed to be commented out, still?
File asterix-metadata/src/main/java/edu/uci/ics/asterix/metadata/entitytupletranslators/

Line 148:                 }
Do we actually want to totally remove UNION type?

Line 378:         } catch (Exception e) {
Could you elaborate a little more on this TODO?
File asterix-om/src/main/java/edu/uci/ics/asterix/om/typecomputer/impl/

Line 57: 
What's the reason here for the exception thrown rather than return?
File asterix-om/src/main/java/edu/uci/ics/asterix/om/typecomputer/impl/

Line 71: 
Same question r.e. exception rather than return

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I223aded8aaf80f0688358899c0e8b0d6988fac93
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ildar Absalyamov <>
Gerrit-Reviewer: Ian Maxon <>
Gerrit-Reviewer: Jenkins <>
Gerrit-Reviewer: Steven Jacobs <>
Gerrit-HasComments: Yes

View raw message