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/10/16 12:15:47 UTC

svn commit: r1632264 - /jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/FieldFactory.java

Author: chetanm
Date: Thu Oct 16 10:15:46 2014
New Revision: 1632264

URL: http://svn.apache.org/r1632264
Log:
OAK-2204 - Support for changing the precision used for storing the date values

Changing the default to always store date in millis. Later we would make it configurable

Modified:
    jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/FieldFactory.java

Modified: jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/FieldFactory.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/FieldFactory.java?rev=1632264&r1=1632263&r2=1632264&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/FieldFactory.java (original)
+++ jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/FieldFactory.java Thu Oct 16 10:15:46 2014
@@ -17,7 +17,6 @@
 package org.apache.jackrabbit.oak.plugins.index.lucene;
 
 import java.util.Arrays;
-import java.util.concurrent.TimeUnit;
 
 import com.google.common.primitives.Ints;
 import org.apache.jackrabbit.oak.api.Type;
@@ -114,10 +113,8 @@ public final class FieldFactory {
         if( date == null){
             return null;
         }
-        //TODO Should we change the precision to 5 min resolution
-        //TODO make if configurable as part of property definition
-        long millis = ISO8601.parse(date).getTimeInMillis();
-        return TimeUnit.MILLISECONDS.toSeconds(millis);
+        //TODO OAK-2204 - Should we change the precision to lower resolution
+        return ISO8601.parse(date).getTimeInMillis();
     }
 
 }