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

        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.
        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.
        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?
        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.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development