Mailing-List: contact dev-help@metamodel.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@metamodel.incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of i.am.kasper.sorensen@gmail.com
 designates 209.85.219.48 as permitted sender)
MIME-Version: 1.0
Date: Wed, 28 Aug 2013 11:11:31 +0200
Message-ID: 
 <CA+U3tt8=6e3_nroL8a6D2Pw5v-Z6Lrt5=7ZacWxg=Su_JBuGew@mail.gmail.com>
Subject: [PATCH] get rid of deprecated "_items" field in DefaultRow
From: =?ISO-8859-1?Q?Kasper_S=F8rensen?= <i.am.kasper.sorensen@gmail.com>
To: "dev@metamodel.incubator.apache.org" <dev@metamodel.incubator.apache.org>
Content-Type: text/plain; charset=ISO-8859-1

Hi all,

I was debugging some code and ran into a legacy thing in the
DefaultRow where we have a deprecated field, "_items". The role of
this field has been taken over by the "_header" field which has
another type etc., but for backwards compatible deserialization
purposes the old "_items" field is still there.

So I did a bit of investigation on how to get rid of the field
entirely and simply do some conversion on the fly while deserializing
legacy objects. And that seems to work nicely actually. I've created a
patch and posted it as a Gist here:

https://gist.github.com/kaspersorensen/6363895

Any feedback is welcome. And if you agree, can we then add this to the code?

Kasper