gjacoby126 commented on code in PR #1448:
URL: https://github.com/apache/phoenix/pull/1448#discussion_r886071338
##########
phoenix-core/src/it/java/org/apache/phoenix/end2end/AlterTableIT.java:
##########
@@ -325,8 +328,7 @@ public static void verifySchemaExport(PTable newTable,
Configuration conf) throw
//needed in an external schema registry. TODO: fix the base column
count anyway
String baseColumnCountPattern = "(?i)\\s*baseColumnCount:\\s\".*\"";
expectedSchemaText =
expectedSchemaText.replaceAll(baseColumnCountPattern, "");
- actualSchemaText =
expectedSchemaText.replaceAll(baseColumnCountPattern, "");
-
+ actualSchemaText = actualSchemaText.replaceAll(baseColumnCountPattern,
"");
Review Comment:
This was a bug in the test -- it was checking the wrong schema, and so
missed that the physical table names (and salted behavior, after I added the
SALT_BUCKETS above) were wrong
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
