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 405462e  [MINOR] Fix invalid consistency checks of spark 
append_aligned rbind
405462e is described below

commit 405462e84ad1192e447bb09c03fe20d112bf6afb
Author: Matthias Boehm <mboe...@gmail.com>
AuthorDate: Sat May 30 00:39:07 2020 +0200

    [MINOR] Fix invalid consistency checks of spark append_aligned rbind
---
 .../apache/sysds/runtime/instructions/spark/BinarySPInstruction.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/src/main/java/org/apache/sysds/runtime/instructions/spark/BinarySPInstruction.java
 
b/src/main/java/org/apache/sysds/runtime/instructions/spark/BinarySPInstruction.java
index dc4e09b..3c3e5b6 100644
--- 
a/src/main/java/org/apache/sysds/runtime/instructions/spark/BinarySPInstruction.java
+++ 
b/src/main/java/org/apache/sysds/runtime/instructions/spark/BinarySPInstruction.java
@@ -459,7 +459,7 @@ public abstract class BinarySPInstruction extends 
ComputationSPInstruction {
                }
                
                if( checkAligned ) {
-                       if( mc1.getCols() % mc1.getBlocksize() != 0 )
+                       if( (cbind ? mc1.getCols() : mc1.getRows()) % 
mc1.getBlocksize() != 0 )
                                throw new DMLRuntimeException("Input matrices 
are not aligned to blocksize boundaries. Wrong append selected");
                }
        }

Reply via email to