gortiz commented on code in PR #15766:
URL: https://github.com/apache/pinot/pull/15766#discussion_r2087251976
##########
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/index/IndexService.java:
##########
@@ -53,18 +56,42 @@
*/
@ThreadSafe
public class IndexService {
+ private static final Logger LOGGER =
LoggerFactory.getLogger(IndexService.class);
private static volatile IndexService _instance = fromServiceLoader();
private final List<IndexType<?, ?, ?>> _allIndexes;
private final Map<String, IndexType<?, ?, ?>> _allIndexesById;
- private IndexService(Set<IndexPlugin<?>> allPlugins) {
- ImmutableMap.Builder<String, IndexType<?, ?, ?>> builder =
ImmutableMap.builder();
+ public IndexService(Set<IndexPlugin<?>> allPlugins) {
Review Comment:
I've opened this method so anyone can register their own IndexService by
calling the static method IndexService.setInstance, which was open before but
useless given nobody could instanciate an IndexService different than the one
that reads from the classpath
--
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]
