bhalchandrap commented on a change in pull request #2439:
URL: https://github.com/apache/thrift/pull/2439#discussion_r723768281
##########
File path: lib/java/src/org/apache/thrift/TDeserializer.java
##########
@@ -353,4 +418,265 @@ private TField locateField(byte[] bytes, TFieldIdEnum
fieldIdPathFirst, TFieldId
public void fromString(TBase base, String data) throws TException {
deserialize(base, data.getBytes());
}
+
+ // ----------------------------------------------------------------------
+ // Methods related to partial deserialization.
+
+ /**
+ * Deserializes the given serialized blob.
+ *
+ * @param bytes the serialized blob.
+ * @return deserialized instance.
+ * @throws TException if an error is encountered during deserialization.
+ */
+ public Object partialDeserializeObject(byte[] bytes) throws TException {
+ return this.deserialize(bytes, 0, bytes.length);
+ }
+
+ /**
+ * Deserializes the given serialized blob.
+ *
+ * @param bytes the serialized blob.
+ * @param offset the blob is read starting at this offset.
+ * @param length the size of blob read (in number of bytes).
+ * @return deserialized instance.
+ * @throws TException if an error is encountered during deserialization.
+ */
+ public Object deserialize(byte[] bytes, int offset, int length) throws
TException {
+ ensurePartialDeserializationMode();
Review comment:
sorry that was an oversight. The method should have been called
partialDeserializeObject() like its other overload. fixed. That should address
both #1 and #2 points above. The name clarifies two intents:
1. it performs partial deserialization
2. it returns an Object
--
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]
