Github user karanmehta93 commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/359#discussion_r223897941
--- Diff:
phoenix-core/src/main/java/org/apache/phoenix/query/QueryConstants.java ---
@@ -357,18 +357,28 @@
+ ColumnFamilyDescriptorBuilder.KEEP_DELETED_CELLS +
"=%s,\n" + PhoenixDatabaseMetaData.TRANSACTIONAL + "="
+ Boolean.FALSE;
- public static final String CREATE_MUTEX_METADTA =
- "CREATE IMMUTABLE TABLE " + SYSTEM_CATALOG_SCHEMA + ".\"" +
SYSTEM_MUTEX_TABLE_NAME + "\"(\n" +
- // Pk columns
- TENANT_ID + " VARCHAR NULL," +
- TABLE_SCHEM + " VARCHAR NULL," +
- TABLE_NAME + " VARCHAR NOT NULL," +
- COLUMN_NAME + " VARCHAR NULL," + // null for table row
- COLUMN_FAMILY + " VARCHAR NULL " + // using for CF to
uniqueness for columns
- "CONSTRAINT " + SYSTEM_TABLE_PK_NAME + " PRIMARY KEY (" +
TENANT_ID + ","
- + TABLE_SCHEM + "," + TABLE_NAME + "," + COLUMN_NAME + ","
+ COLUMN_FAMILY + "))\n" +
- HConstants.VERSIONS + "=%s,\n" +
- ColumnFamilyDescriptorBuilder.KEEP_DELETED_CELLS + "=%s,\n"
+
- PhoenixDatabaseMetaData.TRANSACTIONAL + "=" + Boolean.FALSE;
-
+ public static final String CREATE_MUTEX_METADTA =
--- End diff --
sure, there is some indentation difference between 364-365, would be good
to align them as well.
---
