somandal commented on code in PR #8953:
URL: https://github.com/apache/pinot/pull/8953#discussion_r909062567
##########
pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/index/mutable/MutableForwardIndex.java:
##########
@@ -92,11 +92,26 @@ default int getDictIdMV(int docId, int[] dictIdBuffer) {
throw new UnsupportedOperationException();
}
+ /**
+ * Reads the dictionary ids for a multi-value column at the given document
id into a buffer and returns the buffer.
+ *
+ * @param docId Document id
+ * @return A buffer containing the multi-value entries
+ */
+ default int[] getDictIdMV(int docId) {
Review Comment:
So I didn't see a need to change the setDictIdMV() API as it already takes
an array of dictIDs and doesn't need to return anything:
```
default void setDictIdMV(int docId, int[] dictIds) {
throw new UnsupportedOperationException();
}
```
The above is implemented by only `FixedByteMVMutableForwardIndex` today.
--
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
