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 ju...@apache.org on 2014/03/18 20:06:50 UTC

svn commit: r1579017 - /jackrabbit/oak/trunk/oak-upgrade/src/main/java/org/apache/jackrabbit/oak/upgrade/JackrabbitNodeState.java

Author: jukka
Date: Tue Mar 18 19:06:50 2014
New Revision: 1579017

URL: http://svn.apache.org/r1579017
Log:
OAK-1565: Optimize repository upgrade by loading bundles where possible

DATE values are already stored as strings, so no need to roundtrip them through Calendar

Modified:
    jackrabbit/oak/trunk/oak-upgrade/src/main/java/org/apache/jackrabbit/oak/upgrade/JackrabbitNodeState.java

Modified: jackrabbit/oak/trunk/oak-upgrade/src/main/java/org/apache/jackrabbit/oak/upgrade/JackrabbitNodeState.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-upgrade/src/main/java/org/apache/jackrabbit/oak/upgrade/JackrabbitNodeState.java?rev=1579017&r1=1579016&r2=1579017&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-upgrade/src/main/java/org/apache/jackrabbit/oak/upgrade/JackrabbitNodeState.java (original)
+++ jackrabbit/oak/trunk/oak-upgrade/src/main/java/org/apache/jackrabbit/oak/upgrade/JackrabbitNodeState.java Tue Mar 18 19:06:50 2014
@@ -266,7 +266,7 @@ class JackrabbitNodeState extends Abstra
                     name, value.getBoolean(), Type.BOOLEAN);
         case PropertyType.DATE:
             return PropertyStates.createProperty(
-                    name, ISO8601.format(value.getCalendar()), Type.DATE);
+                    name, value.getString(), Type.DATE);
         case PropertyType.DECIMAL:
             return PropertyStates.createProperty(
                     name, value.getDecimal(), Type.DECIMAL);
@@ -318,7 +318,7 @@ class JackrabbitNodeState extends Abstra
         case PropertyType.DATE:
             List<String> dates = newArrayListWithCapacity(values.length);
             for (InternalValue value : values) {
-                dates.add(ISO8601.format(value.getCalendar()));
+                dates.add(value.getString());
             }
             return PropertyStates.createProperty(name, dates, Type.DATES);
         case PropertyType.DECIMAL: