You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by ch...@apache.org on 2014/07/29 09:20:59 UTC
svn commit: r1614272 -
/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/property/PropertyIndexEditor.java
Author: chetanm
Date: Tue Jul 29 07:20:59 2014
New Revision: 1614272
URL: http://svn.apache.org/r1614272
Log:
OAK-1997 - Improve the message in UniquenessConstraintViolation exception to provide more details
Include path and propertyName in exception messages
Modified:
jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/property/PropertyIndexEditor.java
Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/property/PropertyIndexEditor.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/property/PropertyIndexEditor.java?rev=1614272&r1=1614271&r2=1614272&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/property/PropertyIndexEditor.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/index/property/PropertyIndexEditor.java Tue Jul 29 07:20:59 2014
@@ -277,9 +277,10 @@ class PropertyIndexEditor implements Ind
IndexStoreStrategy s = getStrategy(true);
for (String key : keysToCheckForUniqueness) {
if (s.count(indexMeta, singleton(key), 2) > 1) {
+ String msg = String.format("Uniqueness constraint violated at path [%s] for one of the " +
+ "property in %s having value %s", getPath(), propertyNames, key);
throw new CommitFailedException(
- CONSTRAINT, 30,
- "Uniqueness constraint violated for key " + key);
+ CONSTRAINT, 30, msg);
}
}
}