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();
}
}