[systemml] branch master updated: [SYSTEMML-2511] Fix bitset sparsity estimation on large input data

[mboehm7]

This is an automated email from the ASF dual-hosted git repository.

mboehm7 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/systemml.git


The following commit(s) were added to refs/heads/master by this push:
     new fe83cad  [SYSTEMML-2511] Fix bitset sparsity estimation on large input 
data
fe83cad is described below

commit fe83cad3e13d049eacea19662b1a4e3b1704cb6d
Author: Matthias Boehm <mboe...@gmail.com>
AuthorDate: Tue Feb 19 15:08:08 2019 +0100

    [SYSTEMML-2511] Fix bitset sparsity estimation on large input data
    
    This patch fixes a corruption introduced by previous refactoring that
    led to always allocating a BitsetMatrix1 (w/ linearized long array)
    independent of the input size, leading to incorrect sketches and class
    cast exceptions on subsequent estimation.
---
 src/main/java/org/apache/sysml/hops/estim/EstimatorBitsetMM.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/org/apache/sysml/hops/estim/EstimatorBitsetMM.java 
b/src/main/java/org/apache/sysml/hops/estim/EstimatorBitsetMM.java
index e26dd49..cf9f627 100644
--- a/src/main/java/org/apache/sysml/hops/estim/EstimatorBitsetMM.java
+++ b/src/main/java/org/apache/sysml/hops/estim/EstimatorBitsetMM.java
@@ -88,7 +88,7 @@ public class EstimatorBitsetMM extends SparsityEstimator
                        return null;
                //ensure synopsis is properly cached and reused
                if( node.isLeaf() && node.getSynopsis() == null )
-                       node.setSynopsis(new BitsetMatrix1(node.getData()));
+                       node.setSynopsis(createBitset(node.getData()));
                else if( !node.isLeaf() )
                        estim(node); //recursively obtain synopsis
                return (BitsetMatrix) node.getSynopsis();