Copilot commented on code in PR #15900:
URL: https://github.com/apache/pinot/pull/15900#discussion_r2121652672
##########
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java:
##########
@@ -466,6 +470,23 @@ private boolean verifyTableType(String tableName,
TableType tableType, TableType
return typeFromTableName == null || typeFromTableName == expectedType;
}
+ private void validateLogicalTableReference(String tableName) {
Review Comment:
[nitpick] The logical table reference validation logic in this resource is
similar to that in TableConfigsRestletResource; consider centralizing this
logic into a shared utility method to reduce duplication.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
