noob-se7en commented on code in PR #19468:
URL: https://github.com/apache/pinot/pull/19468#discussion_r4087123309
##########
pinot-server/src/main/java/org/apache/pinot/server/starter/helix/HelixInstanceDataManager.java:
##########
@@ -355,7 +379,6 @@ private TableDataManager createTableDataManager(String
tableNameWithType) {
Preconditions.checkState(tableCreationTimeMs > tableDeleteTimeMs,
"Table: %s was recently deleted (deleted %dms ago) but the table
config was created before that (created "
Review Comment:
not from this PR, but this guard now also fires for tables that never had a
local manager so it will show up more: guava `checkState` only substitutes
`%s`, so the `%dms ago` placeholders render literally and the two durations get
appended in brackets at the end of the message.
--
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]
