Jackie-Jiang commented on code in PR #16349:
URL: https://github.com/apache/pinot/pull/16349#discussion_r2211107421
##########
pinot-plugins/pinot-input-format/pinot-json/src/main/java/org/apache/pinot/plugin/inputformat/json/JSONMessageDecoder.java:
##########
@@ -67,9 +66,4 @@ public GenericRow decode(byte[] payload, GenericRow
destination) {
return null;
}
}
-
Review Comment:
I was trying to deprecate the usage of decode with offset/length because I
don't see it possible with consumers supported in OSS, but good point we can
try to eliminate copy as much as possible before deprecating it
--
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
