walterddr commented on code in PR #8980:
URL: https://github.com/apache/pinot/pull/8980#discussion_r908882798
##########
pinot-server/src/main/java/org/apache/pinot/server/starter/helix/BaseServerStarter.java:
##########
@@ -183,8 +183,9 @@ public void init(PinotConfiguration serverConf)
int dataTableVersion =
_serverConf.getProperty(Server.CONFIG_OF_CURRENT_DATA_TABLE_VERSION,
Server.DEFAULT_CURRENT_DATA_TABLE_VERSION);
if (dataTableVersion > Server.DEFAULT_CURRENT_DATA_TABLE_VERSION) {
- throw new UnsupportedOperationException("Setting experimental DataTable
version newer than default via config "
- + "is not allowed. Current default DataTable version: " +
Server.DEFAULT_CURRENT_DATA_TABLE_VERSION);
+ LOGGER.warn("Setting experimental DataTable version newer than default
via config could result in"
+ + " backward-compatibility issues. Current default DataTable
version: "
+ + Server.DEFAULT_CURRENT_DATA_TABLE_VERSION);
Review Comment:
We need to allow enabling newer default DATA_TABLE_VERSION because
multi-stage engine is not backward compatible with V3 and older data table
--
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]
