Avro
  1. Avro
  2. AVRO-33

C primitives for Avro (de)serialization

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0.0
    • Component/s: c
    • Labels:
      None
    • Environment:

      Tested on Linux CentOS 5.3 and MacOS X. MacOS X fails one of the five unit tests.

      Description

      This is a patch to being adding C binding to Avro.

      See src/c/README for more details.

        Activity

        Matt Massie created issue -
        Hide
        Matt Massie added a comment -

        This patch will add ant targets for compiling, testing and packaging a C library for (de)serializing Avro primitives.

        Show
        Matt Massie added a comment - This patch will add ant targets for compiling, testing and packaging a C library for (de)serializing Avro primitives.
        Matt Massie made changes -
        Field Original Value New Value
        Status Open [ 1 ] Patch Available [ 10002 ]
        Matt Massie made changes -
        Attachment PATCH-c-primitives.patch [ 12408103 ]
        Hide
        Matt Massie added a comment -

        We may want to add a C component to Jira in addition to the current Java and Python components for me to filing this work under. I just filed this under Component: None for now.

        Show
        Matt Massie added a comment - We may want to add a C component to Jira in addition to the current Java and Python components for me to filing this work under. I just filed this under Component: None for now.
        Hide
        Sharad Agarwal added a comment -

        I just created the 'c' component.

        Show
        Sharad Agarwal added a comment - I just created the 'c' component.
        Sharad Agarwal made changes -
        Component/s c [ 12312857 ]
        Hide
        Matt Massie added a comment -

        Thanks Sharad!

        Show
        Matt Massie added a comment - Thanks Sharad!
        Hide
        Doug Cutting added a comment -

        This looks like a great start to C support for Avro!

        One question: Do we need a separate COPYING, NEWS or ChangeLog here? We'll probably always distribute this with the rest of Avro, which already has equivalent files, no?

        Show
        Doug Cutting added a comment - This looks like a great start to C support for Avro! One question: Do we need a separate COPYING, NEWS or ChangeLog here? We'll probably always distribute this with the rest of Avro, which already has equivalent files, no?
        Doug Cutting made changes -
        Assignee Matt Massie [ massie ]
        Hide
        Matt Massie added a comment -

        The files...

        • INSTALL
        • NEWS
        • README
        • AUTHORS
        • ChangeLog
        • COPYING
        • depcomp
        • missing

        are require by automake. In the future, I could just get around the issue by using relative symlinks for those files (I think).

        Show
        Matt Massie added a comment - The files... INSTALL NEWS README AUTHORS ChangeLog COPYING depcomp missing are require by automake . In the future, I could just get around the issue by using relative symlinks for those files (I think).
        Hide
        Doug Cutting added a comment -

        I just committed this. Thanks, Matt!

        I look forward to seeing the rest of the C implementation.

        Show
        Doug Cutting added a comment - I just committed this. Thanks, Matt! I look forward to seeing the rest of the C implementation.
        Doug Cutting made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Fix Version/s 1.0 [ 12313858 ]
        Resolution Fixed [ 1 ]
        Doug Cutting made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        1m 1s 1 Matt Massie 14/May/09 06:38
        Patch Available Patch Available Resolved Resolved
        15h 37m 1 Doug Cutting 14/May/09 22:15
        Resolved Resolved Closed Closed
        61d 1h 34m 1 Doug Cutting 14/Jul/09 23:50

          People

          • Assignee:
            Matt Massie
            Reporter:
            Matt Massie
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development