Fixed.

On 02/06/2013 03:15 PM, James Taylor wrote:
>
> In src/main/java/com/salesforce/phoenix/schema/MetaDataClient.java:
>
> > @@ -252,12 +258,13 @@ public MutationState createTable(CreateTableStatement statement, byte[][] splits
> >              Map<String,Object> tableProps = Collections.emptyMap();
> >              if (!statement.getProps().isEmpty()) {
> >                  if (statement.isView()) {
> > -                    throw new SQLException("Properties may not be defined for a VIEW");
> > +                    throw SQLExceptionInfo.getNewInfoObject(SQLExceptionCodeEnum.VIEW_WITH_PROPERTIES).genExceptionObject();
>
> Minor consistency nit here: above you name the enum 
> NO_BINARY_IN_ROW_KEY, but here you name it VIEW_WITH_PROPERTIES. I 
> think one way or another we should be consistent. Maybe drop to NO_, 
> since all of these enums are error conditions?
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/forcedotcom/phoenix/pull/35/files#r2919284>.
>