Uploaded image for project: 'Crunch'
  1. Crunch
  2. CRUNCH-612

Add support of private constructors to Avros.reflect

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.15.0
    • Component/s: Core
    • Labels:
      None
    • Flags:
      Patch

      Description

      While Avro reflect supports private no-arg constructors since at least 1.4.0, see this test case, Crunch mandates this constructor to be public for deep copy. It seems like an unfortunate shortcoming rather than a conscious choice.

      Since hiding the mandatory no-arg constructor can be quite useful to avoid API pollution, let's patch AvroDeepCopier to add support of private ctor.

        Attachments

        1. CRUNCH-612.patch
          5 kB
          Clément MATHIEU

          Activity

            People

            • Assignee:
              jwills Josh Wills
              Reporter:
              clement@unportant.info Clément MATHIEU
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: