Code review comment for lp:~mmcm/akiban-server/sql-more-types

Revision history for this message
Nathan Williams (nwilliams) wrote :

Your tradeoffs make sense to me.

There's a workaround TODO on TableDDL.java#285. Can that be removed after your associated parser change?

Looks fine otherwise.

review: Needs Information

« Back to merge proposal