Till Westmann 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 types.
......................................................................


Patch Set 6: Code-Review-1

It seems that there are 2 ways to express nullability in the metadata:
1) using an IsNullable field
2) using a "?" after a type name (this is handled in IndexTupleTranslator.java).
It would be nice if we could make this consistent while we are doing a  
metadata change (to keep the number of such changes small).

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/323
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I223aded8aaf80f0688358899c0e8b0d6988fac93
Gerrit-PatchSet: 6
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ildar Absalyamov <[email protected]>
Gerrit-Reviewer: Ian Maxon <[email protected]>
Gerrit-Reviewer: Ildar Absalyamov <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Carey <[email protected]>
Gerrit-Reviewer: Steven Jacobs <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
Gerrit-HasComments: No

Reply via email to