You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by "Thiruvalluvan M. G. (JIRA)" <ji...@apache.org> on 2010/12/13 18:06:01 UTC

[jira] Issue Comment Edited: (AVRO-711) JSON encoder and decoder for C++

    [ https://issues.apache.org/jira/browse/AVRO-711?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12970896#action_12970896 ] 

Thiruvalluvan M. G. edited comment on AVRO-711 at 12/13/10 12:04 PM:
---------------------------------------------------------------------

This patch implements the JSON encoder and decoder for C++. The design is very similar to that of Java implementation. The patch is incomplete in two respects:
- The decoder does not resolve schema. The Java implementation extends validating decoder to support resolution. I'll submit that as a separate patch.
- The decoder and encoder do not handle recursive schema. They throw when a recursive schema is encountered. Again, I'll address this separately.

Instead of modifying the existing interface, I added two new interfaces Encoder and Decoder in the same lines as Java. So none of the existing applications will have trouble.

The new implementation uses the buffers and validSchema which we already have.

As a side benefit there are binary encoder/decoder and validating encoder/decoder using the new interface.

The new implementation is rigorously tested using the same test cases as that of Java.

This patch may have conflicts with AVRO-700. I'll take care of them once AVRO-700 is done.

      was (Author: thiru_mg):
    This patch implements the JSON encoder and decoder for C++. The design is very similar to that of Java implementation. The patch is incomplete in two respects:
- The decoder does not resolve schema. The Java implementation extends validating decoder to support resolution. I'll submit that as a separate patch.
- The decoder and encoder do not handle recursive schema. They throw when a recursive schema is encountered. Again, I'll address this separately.

Instead of modifying the existing interface, I added two new interfaces Encoder and Decoder in the same lines as Java. So none of the existing applications will have trouble.

The new implementation uses the buffers and validSchema which we already have.

As a side benefit there are binary encoder/decoder and validating encoder/decoder using the new interface.

The new implementation is rigorously tested using the same test cases as that of Java.

This patch may have conflicts with Avro-700. I'll take care of them once Avro-700 is done.
  
> JSON encoder and decoder for C++
> --------------------------------
>
>                 Key: AVRO-711
>                 URL: https://issues.apache.org/jira/browse/AVRO-711
>             Project: Avro
>          Issue Type: Improvement
>          Components: c++
>    Affects Versions: 1.5.0
>         Environment: Linux, Cygwin
>            Reporter: Thiruvalluvan M. G.
>            Assignee: Thiruvalluvan M. G.
>         Attachments: AVRO-711.patch
>
>
> Forthcoming patch implements JSON encoder and decoder for C++.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.