crunch-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Clément MATHIEU (JIRA) <j...@apache.org>
Subject [jira] [Created] (CRUNCH-612) Add support of private constructors to Avros.reflect
Date Tue, 19 Jul 2016 19:01:20 GMT
Clément MATHIEU created CRUNCH-612:
--------------------------------------

             Summary: Add support of private constructors to Avros.reflect 
                 Key: CRUNCH-612
                 URL: https://issues.apache.org/jira/browse/CRUNCH-612
             Project: Crunch
          Issue Type: Improvement
          Components: Core
            Reporter: Clément MATHIEU
            Assignee: Josh Wills


While Avro reflect supports private no-arg constructors since at least 1.4.0, see [this test
case|https://gist.github.com/cykl/bf8936fca0410d0cea61402bf5b1e5f9], 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. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message