jackrabbit-oak-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jukka Zitting <jukka.zitt...@gmail.com>
Subject Re: svn commit: r1298002 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/mk/store/StoredNodeAsState.java
Date Fri, 09 Mar 2012 14:12:28 GMT


On Thu, Mar 8, 2012 at 12:47 AM, Michael Dürig <mduerig@apache.org> wrote:
> Right. I should have read the Javadoc ;-) However, I'd make it more explicit
> there, that sub classes must not refine equality (i.e. take into account
> values of other fields). This will in almost any case lead to a broken
> contract. For example (emphasis added): "Two property states are considered
> equal if *and only if* both their names and encoded values match."

Good point. Done in revision 1298833.

On a related note, we probably need to think about value encoding at
some point. Are we happy with having *all* property values persisted
as strings (or parsed/re-formatted at the storage layer), or should we
allow native formatting at least for basic types like numbers and
booleans? A related question is handling of value arrays for
multivalued properties.


Jukka Zitting

View raw message