Github user meiercaleb commented on a diff in the pull request:
https://github.com/apache/incubator-rya/pull/149#discussion_r134583614
--- Diff:
extras/indexing/src/main/java/org/apache/rya/indexing/accumulo/temporal/AccumuloTemporalIndexer.java
---
@@ -158,16 +167,20 @@ public Configuration getConf() {
private void storeStatement(final Statement statement) throws
IOException, IllegalArgumentException {
// if the predicate list is empty, accept all predicates.
// Otherwise, make sure the predicate is on the "valid" list
- final boolean isValidPredicate = validPredicates.isEmpty() ||
validPredicates.contains(statement.getPredicate());
+ final boolean isValidPredicate = validPredicates == null ||
validPredicates.isEmpty() || validPredicates.contains(statement.getPredicate());
if (!isValidPredicate || !(statement.getObject() instanceof
Literal)) {
return;
}
+
final DateTime[] indexDateTimes = new DateTime[2]; // 0 begin, 1
end of interval
extractDateTime(statement, indexDateTimes);
if (indexDateTimes[0]==null) {
return;
}
+ if (!this.isInit)
--- End diff --
Why not just call init at the beginning of this method to ensure that
everything has been initialized?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
