1. for the column name length, we should use 128, since that's the length declared for MFieldSchema.FNAME in metastore/src/model/package.jdo trunk/jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveDatabaseMetaData.java Action: The class property maxColumnNameLength is set to 128. A comment was added to explain why it is 128. 2. for PreparedStatement.execute, you can get rid of the TODO comment; the behavior is correct as is (the true/false return only discriminates between a query with a cursor vs DDL/DML; it's not related to whether any rows are actually returned by the query) Action: TODO was removed.