[ 
https://issues.apache.org/jira/browse/THRIFT-1298?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13093051#comment-13093051
 ] 

Hudson commented on THRIFT-1298:
--------------------------------

Integrated in Thrift #241 (See [https://builds.apache.org/job/Thrift/241/])
    THRIFT-1298. java: Standard scheme doesn't read or write metadata along 
with field values

This patch makes StandardScheme write metadata, and adds tests against a static 
known-good Binary-serialized struct for a reference.

Patch: Armaan Sarkar

bryanduxbury : http://svn.apache.org/viewvc/?view=rev&rev=1162917
Files : 
* /thrift/trunk/compiler/cpp/src/generate/t_java_generator.cc
* /thrift/trunk/lib/java/test/org/apache/thrift/Fixtures.java
* /thrift/trunk/lib/java/test/org/apache/thrift/scheme
* /thrift/trunk/lib/java/test/org/apache/thrift/scheme/TestStandardScheme.java


> Standard scheme doesn't read or write metadata along with field values
> ----------------------------------------------------------------------
>
>                 Key: THRIFT-1298
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1298
>             Project: Thrift
>          Issue Type: Bug
>          Components: Java - Compiler
>    Affects Versions: 0.8
>            Reporter: Armaan Sarkar
>            Assignee: Armaan Sarkar
>             Fix For: 0.8
>
>         Attachments: THRIFT-1289_v1.patch
>
>   Original Estimate: 48h
>  Remaining Estimate: 48h
>
> Metadata (such as field type, length, etc) is not read or written to by the 
> standard scheme. This is not the intended functionality. It causes problems 
> reading structs that were written before pluggable serializers came into 
> effect. Since the standard scheme is the default serializer it must function 
> exactly like thrift serialization did in the past (complete with metadata 
> surrounding the fields).

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to