diff --git a/src/main/java/org/apache/sysds/runtime/frame/data/columns/OptionalArray.java b/src/main/java/org/apache/sysds/runtime/frame/data/columns/OptionalArray.java index 9d79fdf297d..ba0197a6aa0 100644 --- a/src/main/java/org/apache/sysds/runtime/frame/data/columns/OptionalArray.java +++ b/src/main/java/org/apache/sysds/runtime/frame/data/columns/OptionalArray.java @@ -22,7 +22,6 @@ import java.io.DataInput; import java.io.DataOutput; import java.io.IOException; -import java.util.HashMap; import java.util.Map; import org.apache.sysds.common.Types.ValueType; @@ -476,56 +475,56 @@ public boolean possiblyContainsNaN() { // @Override // @SuppressWarnings("unchecked") // protected Map createRecodeMap(int estimate) { - // if(getValueType() == ValueType.BOOLEAN) { - // // shortcut for boolean arrays, since we only - // // need to encounter the first two false and true values. - // Map map = new HashMap<>(estimate); - // long id = 1; - // for(int i = 0; i < size() && id <= 2; i++) - // id = addValRecodeMap(map, id, i); - - // return map; - // } - // else if(getValueType() == ValueType.HASH32){ - // Map map = new HashMap<>(estimate); - // HashIntegerArray b = (HashIntegerArray)_a; - // long id = 1; - // for(int i = 0; i < size(); i++){ - // if(_n.get(i)) - // id = b.addValRecodeMap(map, id, i); - // } - // return (Map)map; - // } - // else if(getValueType() == ValueType.HASH64){ - // Map map = new HashMap<>(estimate); - // HashLongArray b = (HashLongArray)_a; - // long id = 1; - // for(int i = 0; i < size(); i++){ - // if(_n.get(i)) - // id = b.addValRecodeMap(map, id, i); - // } - // return (Map)map; - // } - // else - // return super.createRecodeMap(estimate); + // if(getValueType() == ValueType.BOOLEAN) { + // // shortcut for boolean arrays, since we only + // // need to encounter the first two false and true values. + // Map map = new HashMap<>(estimate); + // long id = 1; + // for(int i = 0; i < size() && id <= 2; i++) + // id = addValRecodeMap(map, id, i); + + // return map; + // } + // else if(getValueType() == ValueType.HASH32){ + // Map map = new HashMap<>(estimate); + // HashIntegerArray b = (HashIntegerArray)_a; + // long id = 1; + // for(int i = 0; i < size(); i++){ + // if(_n.get(i)) + // id = b.addValRecodeMap(map, id, i); + // } + // return (Map)map; + // } + // else if(getValueType() == ValueType.HASH64){ + // Map map = new HashMap<>(estimate); + // HashLongArray b = (HashLongArray)_a; + // long id = 1; + // for(int i = 0; i < size(); i++){ + // if(_n.get(i)) + // id = b.addValRecodeMap(map, id, i); + // } + // return (Map)map; + // } + // else + // return super.createRecodeMap(estimate); // } - @Override - public void setM(Map map, AMapToData m, int i){ - _a.setM(map,m,i); + @Override + public void setM(Map map, AMapToData m, int i) { + _a.setM(map, m, i); } - @Override + @Override public void setM(Map map, int si, AMapToData m, int i) { if(_n.get(i)) - _a.setM(map,si,m,i); + _a.setM(map, si, m, i); else m.set(i, si); } - @Override + @Override protected long addValRecodeMap(Map map, long id, int i) { - + if(_n.get(i)) { id = _a.addValRecodeMap(map, id, i); }