[
https://issues.apache.org/jira/browse/JCR-1272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12552774
]
Marcel Reutegger commented on JCR-1272:
---------------------------------------
You are right. The funny thing is getString() already uses getValue(). :-)
I think we should get rid of the sections documented with:
// avoid unnecessary object creation if possible
They add a significant amount of code and I doubt it is worth it.
> Property.getValue() throws RepositoryException with internal error
> ------------------------------------------------------------------
>
> Key: JCR-1272
> URL: https://issues.apache.org/jira/browse/JCR-1272
> Project: Jackrabbit
> Issue Type: Bug
> Components: jackrabbit-core
> Reporter: Marcel Reutegger
> Priority: Minor
> Attachments: JCR-1272.patch
>
>
> Running ConcurrentReadWriteTest (NUM_NODES=5, NUM_THREADS=3,
> RUN_NUM_SECONDS=120) resulted in a RepositoryException calling
> Property.getValue():
> javax.jcr.RepositoryException: Internal error while retrieving value of
> b3fc1ea8-3364-4236-bcc7-dea0baf90640/{}test: null: null
> Debugging shows that it is a NullPointerException:
> java.lang.NullPointerException
> at
> org.apache.jackrabbit.core.PropertyImpl.getValue(PropertyImpl.java:481)
> at
> org.apache.jackrabbit.core.ConcurrentReadWriteTest$1$1.execute(ConcurrentReadWriteTest.java:68)
> at
> org.apache.jackrabbit.core.AbstractConcurrencyTest$Executor.run(AbstractConcurrencyTest.java:110)
> at java.lang.Thread.run(Thread.java:619)
> It's probably the state which has been discarded after the sanityCheck().
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
