bhalchandrap commented on a change in pull request #2439:
URL: https://github.com/apache/thrift/pull/2439#discussion_r723769384
##########
File path: lib/java/src/org/apache/thrift/TDeserializer.java
##########
@@ -61,6 +78,54 @@ public TDeserializer(TProtocolFactory protocolFactory)
throws TTransportExceptio
protocol_ = protocolFactory.getProtocol(trans_);
}
+ /**
+ * Construct a new TDeserializer that supports partial deserialization
+ * that outputs instances of type controlled by the given {@code processor}.
+ *
+ * @param thriftClass a TBase derived class.
+ * @param fieldNames list of fields to deserialize.
+ * @param processor the Processor that handles deserialized field values.
+ * @param protocolFactory the Factory to create a protocol.
+ */
+ public TDeserializer(
Review comment:
Your observation is correct and is intentional. I did not want to change
the behavior of existing methods as it can cause backward compatibility issues.
With that in mind, if one calls any of the existing deserialize() methods,
they will continue to get the existing behavior. One will need to call
partialDeserilizeObject() to perform partial deserialization.
If you strongly believe that we need to change the behavior of existing
methods, I can make it happen. Let me know what you think.
--
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]
