From 417ee4061e81df596a5ef436d69c5bedc51b28d7 Mon Sep 17 00:00:00 2001 From: Robin Bygrave Date: Fri, 21 Aug 2015 21:06:33 +1200 Subject: [PATCH] #395 - No functional change - cleanup unused BeanDelta --- .../server/cluster/BinaryMessage.java | 2 - .../cluster/PacketTransactionEvent.java | 11 +-- .../server/transaction/BeanDelta.java | 98 ------------------- .../server/transaction/BeanDeltaList.java | 42 -------- .../server/transaction/BeanDeltaMap.java | 48 --------- .../server/transaction/BeanDeltaProperty.java | 36 ------- .../server/transaction/BeanPersistIds.java | 2 +- .../transaction/RemoteTransactionEvent.java | 33 ------- 8 files changed, 4 insertions(+), 268 deletions(-) delete mode 100644 src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDelta.java delete mode 100644 src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDeltaList.java delete mode 100644 src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDeltaMap.java delete mode 100644 src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDeltaProperty.java diff --git a/src/main/java/com/avaje/ebeaninternal/server/cluster/BinaryMessage.java b/src/main/java/com/avaje/ebeaninternal/server/cluster/BinaryMessage.java index 72ed1f323..2883bd3ca 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/cluster/BinaryMessage.java +++ b/src/main/java/com/avaje/ebeaninternal/server/cluster/BinaryMessage.java @@ -23,8 +23,6 @@ public class BinaryMessage { public static final int TYPE_MSGCONTROL = 0; public static final int TYPE_BEANIUD = 1; public static final int TYPE_TABLEIUD = 2; - public static final int TYPE_BEANDELTA = 3; - public static final int TYPE_BEANPATHUPDATE = 4; public static final int TYPE_MSGACK = 8; public static final int TYPE_MSGRESEND = 9; diff --git a/src/main/java/com/avaje/ebeaninternal/server/cluster/PacketTransactionEvent.java b/src/main/java/com/avaje/ebeaninternal/server/cluster/PacketTransactionEvent.java index 5f44c23cd..d6a408351 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/cluster/PacketTransactionEvent.java +++ b/src/main/java/com/avaje/ebeaninternal/server/cluster/PacketTransactionEvent.java @@ -1,14 +1,13 @@ package com.avaje.ebeaninternal.server.cluster; -import java.io.DataInput; -import java.io.IOException; - import com.avaje.ebeaninternal.api.SpiEbeanServer; import com.avaje.ebeaninternal.api.TransactionEventTable.TableIUD; -import com.avaje.ebeaninternal.server.transaction.BeanDelta; import com.avaje.ebeaninternal.server.transaction.BeanPersistIds; import com.avaje.ebeaninternal.server.transaction.RemoteTransactionEvent; +import java.io.DataInput; +import java.io.IOException; + /** * A Packet holding TransactionEvent data. *

@@ -57,10 +56,6 @@ public class PacketTransactionEvent extends Packet { event.addTableIUD(TableIUD.readBinaryMessage(dataInput)); break; - case BinaryMessage.TYPE_BEANDELTA: - event.addBeanDelta(BeanDelta.readBinaryMessage(server, dataInput)); - break; - default: throw new RuntimeException("Invalid Transaction msgType "+msgType); } diff --git a/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDelta.java b/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDelta.java deleted file mode 100644 index 21cb60eae..000000000 --- a/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDelta.java +++ /dev/null @@ -1,98 +0,0 @@ -package com.avaje.ebeaninternal.server.transaction; - -import com.avaje.ebean.bean.EntityBean; -import com.avaje.ebeaninternal.api.SpiEbeanServer; -import com.avaje.ebeaninternal.server.cluster.BinaryMessage; -import com.avaje.ebeaninternal.server.cluster.BinaryMessageList; -import com.avaje.ebeaninternal.server.deploy.BeanDescriptor; -import com.avaje.ebeaninternal.server.deploy.BeanProperty; - -import java.io.DataInput; -import java.io.DataOutputStream; -import java.io.IOException; -import java.util.ArrayList; -import java.util.List; - -public class BeanDelta { - - private final List properties; - - private final BeanDescriptor beanDescriptor; - - private final Object id; - - public BeanDelta(BeanDescriptor beanDescriptor, Object id) { - this.beanDescriptor = beanDescriptor; - this.id = id; - this.properties = new ArrayList(); - } - - public BeanDescriptor getBeanDescriptor() { - return beanDescriptor; - } - - public String toString() { - return "BeanDelta[" + beanDescriptor.getName() + ":" + properties + "]"; - } - - public Object getId() { - return id; - } - - public void add(BeanProperty beanProperty, Object value) { - this.properties.add(new BeanDeltaProperty(beanProperty, value)); - } - - public void add(BeanDeltaProperty propertyDelta) { - this.properties.add(propertyDelta); - } - - public void apply(EntityBean bean) { - - for (int i = 0; i < properties.size(); i++) { - properties.get(i).apply(bean); - } - } - - /** - * Read and return a BeanDelta from the binary input. - */ - public static BeanDelta readBinaryMessage(SpiEbeanServer server, DataInput dataInput) throws IOException { - - String descriptorId = dataInput.readUTF(); - BeanDescriptor desc = server.getBeanDescriptorById(descriptorId); - Object id = desc.getIdBinder().readData(dataInput); - BeanDelta bp = new BeanDelta(desc, id); - - int count = dataInput.readInt(); - for (int i = 0; i < count; i++) { - String propName = dataInput.readUTF(); - BeanProperty beanProperty = desc.getBeanProperty(propName); - Object value = beanProperty.getScalarType().readData(dataInput); - bp.add(beanProperty, value); - } - return bp; - } - - /** - * Write this bean delta in binary message format. - */ - public void writeBinaryMessage(BinaryMessageList msgList) throws IOException { - - BinaryMessage m = new BinaryMessage(50); - - DataOutputStream os = m.getOs(); - os.writeInt(BinaryMessage.TYPE_BEANDELTA); - os.writeUTF(beanDescriptor.getDescriptorId()); - - beanDescriptor.getIdBinder().writeData(os, id); - os.writeInt(properties.size()); - - for (int i = 0; i < properties.size(); i++) { - properties.get(i).writeBinaryMessage(m); - } - - os.flush(); - msgList.add(m); - } -} diff --git a/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDeltaList.java b/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDeltaList.java deleted file mode 100644 index d3954da85..000000000 --- a/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDeltaList.java +++ /dev/null @@ -1,42 +0,0 @@ -package com.avaje.ebeaninternal.server.transaction; - -import com.avaje.ebeaninternal.server.cluster.BinaryMessageList; -import com.avaje.ebeaninternal.server.deploy.BeanDescriptor; - -import java.io.IOException; -import java.util.ArrayList; -import java.util.List; - -public class BeanDeltaList { - - private final BeanDescriptor beanDescriptor; - - private final List deltaBeans = new ArrayList(); - - public BeanDeltaList(BeanDescriptor beanDescriptor) { - this.beanDescriptor = beanDescriptor; - } - - public String toString() { - return deltaBeans.toString(); - } - - public BeanDescriptor getBeanDescriptor() { - return beanDescriptor; - } - - public void add(BeanDelta b) { - deltaBeans.add(b); - } - - public List getDeltaBeans() { - return deltaBeans; - } - - public void writeBinaryMessage(BinaryMessageList msgList) throws IOException { - for (int i = 0; i < deltaBeans.size(); i++) { - deltaBeans.get(i).writeBinaryMessage(msgList); - } - } - -} diff --git a/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDeltaMap.java b/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDeltaMap.java deleted file mode 100644 index e5282d049..000000000 --- a/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDeltaMap.java +++ /dev/null @@ -1,48 +0,0 @@ -package com.avaje.ebeaninternal.server.transaction; - -import com.avaje.ebeaninternal.server.deploy.BeanDescriptor; - -import java.util.Collection; -import java.util.HashMap; -import java.util.List; -import java.util.Map; - -public class BeanDeltaMap { - - private final Map deltaMap = new HashMap(); - - public BeanDeltaMap() { - } - - public BeanDeltaMap(List deltaBeans) { - if (deltaBeans != null) { - for (int i = 0; i < deltaBeans.size(); i++) { - BeanDelta deltaBean = deltaBeans.get(i); - addBeanDelta(deltaBean); - } - } - } - - public String toString() { - return deltaMap.values().toString(); - } - - public void addBeanDelta(BeanDelta beanDelta) { - BeanDescriptor d = beanDelta.getBeanDescriptor(); - BeanDeltaList list = getDeltaBeanList(d); - list.add(beanDelta); - } - - public Collection deltaLists() { - return deltaMap.values(); - } - - private BeanDeltaList getDeltaBeanList(BeanDescriptor d) { - BeanDeltaList deltaList = deltaMap.get(d.getFullName()); - if (deltaList == null) { - deltaList = new BeanDeltaList(d); - deltaMap.put(d.getFullName(), deltaList); - } - return deltaList; - } -} diff --git a/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDeltaProperty.java b/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDeltaProperty.java deleted file mode 100644 index 2892832e4..000000000 --- a/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanDeltaProperty.java +++ /dev/null @@ -1,36 +0,0 @@ -package com.avaje.ebeaninternal.server.transaction; - -import com.avaje.ebean.bean.EntityBean; -import com.avaje.ebeaninternal.server.cluster.BinaryMessage; -import com.avaje.ebeaninternal.server.deploy.BeanProperty; - -import java.io.DataOutputStream; -import java.io.IOException; - -public class BeanDeltaProperty { - - private final BeanProperty beanProperty; - - private final Object value; - - public BeanDeltaProperty(BeanProperty beanProperty, Object value) { - this.beanProperty = beanProperty; - this.value = value; - } - - public String toString() { - return beanProperty.getName() + ":" + value; - } - - public void apply(EntityBean bean) { - beanProperty.setValue(bean, value); - } - - public void writeBinaryMessage(BinaryMessage m) throws IOException { - - DataOutputStream os = m.getOs(); - os.writeUTF(beanProperty.getName()); - beanProperty.getScalarType().writeData(os, value); - } - -} diff --git a/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanPersistIds.java b/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanPersistIds.java index aab9156a3..6ef1bdf20 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanPersistIds.java +++ b/src/main/java/com/avaje/ebeaninternal/server/transaction/BeanPersistIds.java @@ -19,7 +19,7 @@ import java.util.List; * Wraps the information representing a Inserted Updated or Deleted Bean. *

* This information is broadcast across the cluster so that remote BeanListeners - * are notified of the inserts updates and deletes that occured. + * are notified of the inserts updates and deletes that occurred. *

*

* You control it the data is broadcast and what data is broadcast by the diff --git a/src/main/java/com/avaje/ebeaninternal/server/transaction/RemoteTransactionEvent.java b/src/main/java/com/avaje/ebeaninternal/server/transaction/RemoteTransactionEvent.java index 94aea43c3..060cfda5c 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/transaction/RemoteTransactionEvent.java +++ b/src/main/java/com/avaje/ebeaninternal/server/transaction/RemoteTransactionEvent.java @@ -14,10 +14,6 @@ public class RemoteTransactionEvent implements Runnable { private List tableList; - private List beanDeltaLists; - - private BeanDeltaMap beanDeltaMap; - private DeleteByIdMap deleteByIdMap; private String serverName; @@ -38,9 +34,6 @@ public class RemoteTransactionEvent implements Runnable { public String toString() { StringBuilder sb = new StringBuilder(); - if (beanDeltaMap != null) { - sb.append(beanDeltaMap); - } sb.append(beanPersistList); if (tableList != null) { sb.append(tableList); @@ -67,12 +60,6 @@ public class RemoteTransactionEvent implements Runnable { beanPersistList.get(i).writeBinaryMessage(msgList); } } - - if (beanDeltaLists != null) { - for (int i = 0; i < beanDeltaLists.size(); i++) { - beanDeltaLists.get(i).writeBinaryMessage(msgList); - } - } } public boolean isEmpty() { @@ -90,20 +77,6 @@ public class RemoteTransactionEvent implements Runnable { tableList.add(tableIud); } - public void addBeanDeltaList(BeanDeltaList deltaList) { - if (beanDeltaLists == null) { - beanDeltaLists = new ArrayList(); - } - beanDeltaLists.add(deltaList); - } - - public void addBeanDelta(BeanDelta beanDelta) { - if (beanDeltaMap == null) { - beanDeltaMap = new BeanDeltaMap(); - } - beanDeltaMap.addBeanDelta(beanDelta); - } - public String getServerName() { return serverName; } @@ -132,10 +105,4 @@ public class RemoteTransactionEvent implements Runnable { return beanPersistList; } - public List getBeanDeltaLists() { - if (beanDeltaMap != null) { - beanDeltaLists.addAll(beanDeltaMap.deltaLists()); - } - return beanDeltaLists; - } }