siddharthteotia commented on code in PR #8499:
URL: https://github.com/apache/pinot/pull/8499#discussion_r847771725
##########
pinot-core/src/main/java/org/apache/pinot/core/common/datatable/DataTableImplV3.java:
##########
@@ -350,32 +340,29 @@ private byte[] serializeMetadata()
* This is to make V3 implementation keep the consumers of Map<String,
String> getMetadata() API in the code happy
* by internally converting it.
*/
- private Map<String, String> deserializeMetadata(byte[] bytes)
+ private Map<String, String> deserializeMetadata(ByteBuffer buffer)
Review Comment:
Can we add a comment / note stating that the caller of this method assumes
that ByteBuffer position is moved accordingly by reads done in this method and
so we should use relative get/read calls on ByteBuffer ?
--
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]
