Mailing-List: contact dev-help@crunch.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@crunch.apache.org
Date: Tue, 19 Jul 2016 19:01:20 +0000 (UTC)
From: =?utf-8?Q?Cl=C3=A9ment_MATHIEU_=28JIRA=29?= <jira@apache.org>
To: crunch-dev@incubator.apache.org
Message-ID: <JIRA.12990870.1468954866000.75206.1468954880774@Atlassian.JIRA>
In-Reply-To: <JIRA.12990870.1468954866000@Atlassian.JIRA>
References: <JIRA.12990870.1468954866000@Atlassian.JIRA> <JIRA.12990870.1468954866357@arcas>
Subject: [jira] [Created] (CRUNCH-612) Add support of private constructors
 to Avros.reflect
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
archived-at: Tue, 19 Jul 2016 19:01:22 -0000

Cl=C3=A9ment MATHIEU created CRUNCH-612:
--------------------------------------

             Summary: Add support of private constructors to Avros.reflect=
=20
                 Key: CRUNCH-612
                 URL: https://issues.apache.org/jira/browse/CRUNCH-612
             Project: Crunch
          Issue Type: Improvement
          Components: Core
            Reporter: Cl=C3=A9ment 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/bf8936fca0410d0cea61402=
bf5b1e5f9], Crunch mandates this constructor to be public for deep copy. It=
 seems like an unfortunate shortcoming rather than a conscious choice.=20

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.=
=20


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