raghavyadav01 commented on code in PR #14609:
URL: https://github.com/apache/pinot/pull/14609#discussion_r1872285256
##########
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/index/map/MapIndexType.java:
##########
@@ -55,6 +55,7 @@ public class MapIndexType extends
AbstractIndexType<MapIndexConfig, MapIndexRead
Collections.singletonList(V1Constants.Indexes.MAP_INDEX_FILE_EXTENSION);
private static final String MAP_INDEX_CREATOR_CLASS_NAME =
"mapIndexCreatorClassName";
private static final String MAP_INDEX_READER_CLASS_NAME =
"mapIndexReaderClassName";
+ private static final String MAP_INDEX_MUTABLE_CLASS_NAME =
"mapIndexMutableClassName";
Review Comment:
Sure. Let me change it, I was just trying to have consistent prefix but I
think we can have MUTABLE_MAP_INDEX_CLASS_NAME = "mutableMapIndexClassName"
--
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]
