You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@systemds.apache.org by ba...@apache.org on 2021/08/26 09:06:33 UTC

[systemds] branch master updated: [MINOR] Remove redundant serialization imports in operators

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

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


The following commit(s) were added to refs/heads/master by this push:
     new eca11c6  [MINOR] Remove redundant serialization imports in operators
eca11c6 is described below

commit eca11c6fe9cff88df2e1960caf1b0cff9bf2b2b6
Author: baunsgaard <ba...@tugraz.at>
AuthorDate: Wed Aug 25 22:38:48 2021 +0200

    [MINOR] Remove redundant serialization imports in operators
    
    Closes #1374
---
 .../apache/sysds/runtime/matrix/operators/AggregateOperator.java   | 5 +----
 .../sysds/runtime/matrix/operators/AggregateTernaryOperator.java   | 4 +---
 .../sysds/runtime/matrix/operators/AggregateUnaryOperator.java     | 3 +--
 .../org/apache/sysds/runtime/matrix/operators/BinaryOperator.java  | 7 ++-----
 .../org/apache/sysds/runtime/matrix/operators/ReorgOperator.java   | 5 +----
 .../org/apache/sysds/runtime/matrix/operators/TernaryOperator.java | 5 +----
 6 files changed, 7 insertions(+), 22 deletions(-)

diff --git a/src/main/java/org/apache/sysds/runtime/matrix/operators/AggregateOperator.java b/src/main/java/org/apache/sysds/runtime/matrix/operators/AggregateOperator.java
index 85ada78..f03d17f 100644
--- a/src/main/java/org/apache/sysds/runtime/matrix/operators/AggregateOperator.java
+++ b/src/main/java/org/apache/sysds/runtime/matrix/operators/AggregateOperator.java
@@ -20,8 +20,6 @@
 
 package org.apache.sysds.runtime.matrix.operators;
 
-import java.io.Serializable;
-
 import org.apache.sysds.common.Types.CorrectionLocationType;
 import org.apache.sysds.runtime.functionobjects.KahanPlus;
 import org.apache.sysds.runtime.functionobjects.KahanPlusSq;
@@ -31,8 +29,7 @@ import org.apache.sysds.runtime.functionobjects.Plus;
 import org.apache.sysds.runtime.functionobjects.ValueFunction;
 
 
-public class AggregateOperator  extends Operator implements Serializable
-{
+public class AggregateOperator extends Operator {
 	private static final long serialVersionUID = 8761527329665129670L;
 
 	public final double initialValue;
diff --git a/src/main/java/org/apache/sysds/runtime/matrix/operators/AggregateTernaryOperator.java b/src/main/java/org/apache/sysds/runtime/matrix/operators/AggregateTernaryOperator.java
index 6214fb4..90f1a04 100644
--- a/src/main/java/org/apache/sysds/runtime/matrix/operators/AggregateTernaryOperator.java
+++ b/src/main/java/org/apache/sysds/runtime/matrix/operators/AggregateTernaryOperator.java
@@ -20,13 +20,11 @@
 
 package org.apache.sysds.runtime.matrix.operators;
 
-import java.io.Serializable;
-
 import org.apache.sysds.runtime.functionobjects.IndexFunction;
 import org.apache.sysds.runtime.functionobjects.ValueFunction;
 
 
-public class AggregateTernaryOperator extends Operator implements Serializable
+public class AggregateTernaryOperator extends Operator
 {
 	private static final long serialVersionUID = 4251745081160216784L;
 	
diff --git a/src/main/java/org/apache/sysds/runtime/matrix/operators/AggregateUnaryOperator.java b/src/main/java/org/apache/sysds/runtime/matrix/operators/AggregateUnaryOperator.java
index a1faae0..532567b 100644
--- a/src/main/java/org/apache/sysds/runtime/matrix/operators/AggregateUnaryOperator.java
+++ b/src/main/java/org/apache/sysds/runtime/matrix/operators/AggregateUnaryOperator.java
@@ -30,8 +30,7 @@ import org.apache.sysds.runtime.functionobjects.ReduceCol;
 import org.apache.sysds.runtime.functionobjects.ReduceRow;
 
 
-public class AggregateUnaryOperator  extends Operator 
-{
+public class AggregateUnaryOperator extends Operator {
 	private static final long serialVersionUID = 6690553323120787735L;
 
 	public final AggregateOperator aggOp;
diff --git a/src/main/java/org/apache/sysds/runtime/matrix/operators/BinaryOperator.java b/src/main/java/org/apache/sysds/runtime/matrix/operators/BinaryOperator.java
index 7cf201a..f0fa36a 100644
--- a/src/main/java/org/apache/sysds/runtime/matrix/operators/BinaryOperator.java
+++ b/src/main/java/org/apache/sysds/runtime/matrix/operators/BinaryOperator.java
@@ -20,8 +20,6 @@
 
 package org.apache.sysds.runtime.matrix.operators;
 
-import java.io.Serializable;
-
 import org.apache.sysds.common.Types.OpOp2;
 import org.apache.sysds.runtime.functionobjects.And;
 import org.apache.sysds.runtime.functionobjects.BitwAnd;
@@ -30,6 +28,7 @@ import org.apache.sysds.runtime.functionobjects.BitwShiftL;
 import org.apache.sysds.runtime.functionobjects.BitwShiftR;
 import org.apache.sysds.runtime.functionobjects.BitwXor;
 import org.apache.sysds.runtime.functionobjects.Builtin;
+import org.apache.sysds.runtime.functionobjects.Builtin.BuiltinCode;
 import org.apache.sysds.runtime.functionobjects.Divide;
 import org.apache.sysds.runtime.functionobjects.Equals;
 import org.apache.sysds.runtime.functionobjects.GreaterThan;
@@ -50,10 +49,8 @@ import org.apache.sysds.runtime.functionobjects.PlusMultiply;
 import org.apache.sysds.runtime.functionobjects.Power;
 import org.apache.sysds.runtime.functionobjects.ValueFunction;
 import org.apache.sysds.runtime.functionobjects.Xor;
-import org.apache.sysds.runtime.functionobjects.Builtin.BuiltinCode;
 
-public class BinaryOperator  extends Operator implements Serializable
-{
+public class BinaryOperator extends Operator {
 	private static final long serialVersionUID = -2547950181558989209L;
 
 	public final ValueFunction fn;
diff --git a/src/main/java/org/apache/sysds/runtime/matrix/operators/ReorgOperator.java b/src/main/java/org/apache/sysds/runtime/matrix/operators/ReorgOperator.java
index 81d7c5c..cc3d2e3 100644
--- a/src/main/java/org/apache/sysds/runtime/matrix/operators/ReorgOperator.java
+++ b/src/main/java/org/apache/sysds/runtime/matrix/operators/ReorgOperator.java
@@ -20,12 +20,9 @@
 
 package org.apache.sysds.runtime.matrix.operators;
 
-import java.io.Serializable;
-
 import org.apache.sysds.runtime.functionobjects.IndexFunction;
 
-public class ReorgOperator  extends Operator implements Serializable
-{
+public class ReorgOperator extends Operator{
 	private static final long serialVersionUID = -5322516429026298404L;
 
 	public final IndexFunction fn;
diff --git a/src/main/java/org/apache/sysds/runtime/matrix/operators/TernaryOperator.java b/src/main/java/org/apache/sysds/runtime/matrix/operators/TernaryOperator.java
index 6ff8e89..92b142d 100644
--- a/src/main/java/org/apache/sysds/runtime/matrix/operators/TernaryOperator.java
+++ b/src/main/java/org/apache/sysds/runtime/matrix/operators/TernaryOperator.java
@@ -20,15 +20,12 @@
 
 package org.apache.sysds.runtime.matrix.operators;
 
-import java.io.Serializable;
-
 import org.apache.sysds.runtime.functionobjects.IfElse;
 import org.apache.sysds.runtime.functionobjects.MinusMultiply;
 import org.apache.sysds.runtime.functionobjects.PlusMultiply;
 import org.apache.sysds.runtime.functionobjects.TernaryValueFunction;
 
-public class TernaryOperator  extends Operator implements Serializable
-{
+public class TernaryOperator extends Operator{
 	private static final long serialVersionUID = 3456088891054083634L;
 	
 	public final TernaryValueFunction fn;