Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.5.1
    • Component/s: c++
    • Labels:
      None

      Description

      The forthcoming patch adds Datafile support in C++.

      1. AVRO-789.patch
        26 kB
        Thiruvalluvan M. G.

        Activity

        Hide
        Doug Cutting added a comment -

        Should we add datafile interop test for C++? This is run from the top-level build.sh. Each language writes one or more data files in build/interop/data/$lang containing randomly generated instances of the share/test/schemas/interop.avsc. Then each language reads the files written by all of the others. It's a rather weak test, but better than nothing.

        Show
        Doug Cutting added a comment - Should we add datafile interop test for C++? This is run from the top-level build.sh. Each language writes one or more data files in build/interop/data/$lang containing randomly generated instances of the share/test/schemas/interop.avsc. Then each language reads the files written by all of the others. It's a rather weak test, but better than nothing.
        Hide
        Thiruvalluvan M. G. added a comment -

        Committed revision 1086866 after adding a couple of more tests and ensuring that the data file is compatible to that used by Avro Java.

        Thank you Doug for reviewing.

        Show
        Thiruvalluvan M. G. added a comment - Committed revision 1086866 after adding a couple of more tests and ensuring that the data file is compatible to that used by Avro Java. Thank you Doug for reviewing.
        Hide
        Thiruvalluvan M. G. added a comment -

        +1 for inclusion in 1.5.1

        Show
        Thiruvalluvan M. G. added a comment - +1 for inclusion in 1.5.1
        Hide
        Doug Cutting added a comment -

        This is great to have! Builds and passes tests for me on Ubuntu 10.10. +1

        It doesn't look like any of the API changes are incompatible, so should we include this in 1.5.1?

        Show
        Doug Cutting added a comment - This is great to have! Builds and passes tests for me on Ubuntu 10.10. +1 It doesn't look like any of the API changes are incompatible, so should we include this in 1.5.1?
        Hide
        Thiruvalluvan M. G. added a comment -

        The patch adds Datafile support for C++.
        Tested on:

        • Cygwin
        • Ubuntu 10.10
        • RHEL 4.8
        Show
        Thiruvalluvan M. G. added a comment - The patch adds Datafile support for C++. Tested on: Cygwin Ubuntu 10.10 RHEL 4.8

          People

          • Assignee:
            Thiruvalluvan M. G.
            Reporter:
            Thiruvalluvan M. G.
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development