gortiz opened a new pull request, #10674: URL: https://github.com/apache/pinot/pull/10674
PR #10623 introduced the new index type concept `prettyName` as a new abstract method. By adding that we break backward compatibility with no reason. Values of `id` and `prettyName` should be the same unless there is some compatibility reason. For example, most indexes created pre `index-spi` used a different name and id in the older `ColumnIndexType` enum. New indexes should be able to have the same `id` and `prettyName`. Therefore this PR changes `getPrettyName` to delegate on `getId` by default -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
