takeseem commented on a change in pull request #1232: Fix hessian2 serialized 
short, byte is converted to int bug
URL: https://github.com/apache/incubator-dubbo/pull/1232#discussion_r181568188
 
 

 ##########
 File path: 
hessian-lite/src/main/java/com/alibaba/com/caucho/hessian/io/AbstractHessianInput.java
 ##########
 @@ -334,12 +335,34 @@ abstract public InputStream readInputStream()
     abstract public Object readObject(Class expectedClass)
             throws IOException;
 
+    /**
+     * Reads an arbitrary object from the input stream.
+     *
+     * @param expectedClass the expected class if the protocol doesn't supply 
it.
+     * @param expectedTypes the runtime type hints, eg: expectedClass equals 
Map, expectedTypes can
+     * equals String.class, Short.class
+     */
+    public Object readObject(Class expectedClass, Class<?>... expectedTypes)
 
 Review comment:
   #1232 
   @lovepoem @zonghaishang 
   the add method will work in Hessian2Input, but in HessianInput not 
unsupported. 
   I think the HessianInput need to change to support the new methods.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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

Reply via email to