Avro
  1. Avro
  2. AVRO-940

C++ design for generic datum could be better.

    Details

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

      Description

      The template functions avro::encode() and avro::decode along with codec_traits has a clean way of encoding and decoding built-in C++ types corresponding to Avro types and types generated by the code generator. However, encoding and decoding GenricDatum is clunky. It uses std::pair and schema objects make makes it hard for programmers to grasp and use. Added complexity is due to use of reference in pair.

      The forthcoming patch simplifies this and makes things uniform.

      1. AVRO-940.patch
        25 kB
        Thiruvalluvan M. G.

        Activity

        Hide
        Thiruvalluvan M. G. added a comment -

        Committed revision 1187383.

        Show
        Thiruvalluvan M. G. added a comment - Committed revision 1187383.

          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