Jackie-Jiang commented on a change in pull request #4555: [Issue #4551] Remove
memory allocation for virtual columns in consumi…
URL: https://github.com/apache/incubator-pinot/pull/4555#discussion_r316949012
##########
File path:
pinot-core/src/main/java/org/apache/pinot/core/indexsegment/IndexSegment.java
##########
@@ -70,16 +70,6 @@
*/
List<StarTreeV2> getStarTrees();
- /**
- * Returns the record for the given document Id.
- * <p>NOTE: don't use this method for high performance code.
- *
- * @param docId Document Id
- * @param reuse Reusable buffer for the record
- * @return Record for the given document Id
- */
- GenericRow getRecord(int docId, GenericRow reuse);
Review comment:
Let's not remove this API as we can use it to read data from segment (both
mutable and immutable)
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
