Github user ejwhite922 commented on a diff in the pull request:

    https://github.com/apache/incubator-rya/pull/269#discussion_r165673213
  
    --- Diff: 
extras/indexing/src/main/java/org/apache/rya/indexing/accumulo/AccumuloIndexingConfiguration.java
 ---
    @@ -287,15 +286,19 @@ public void setAccumuloTemporalPredicates(String[] 
predicates) {
         public String[] getAccumuloTemporalPredicates() {
             return getStrings(ConfigUtils.TEMPORAL_PREDICATES_LIST);
         }
    -    
    -    private static Set<RyaURI> getPropURIFromStrings(String ... props) {
    -        Set<RyaURI> properties = new HashSet<>();
    -        for(String prop: props) {
    -            properties.add(new RyaURI(prop));
    +
    +    private static Set<RyaURI> getPropURIFromStrings(final String... 
props) {
    +        final Set<RyaURI> properties = new HashSet<>();
    +        if (props != null) {
    +            for(final String prop : props) {
    +                if (prop != null) {
    --- End diff --
    
    Added a check for if the prop is blank.  There are unit tests for 
comma-separated values.


---

Reply via email to