[ https://issues.apache.org/jira/browse/GOSSIP-79?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15973945#comment-15973945 ]
ASF GitHub Bot commented on GOSSIP-79: -------------------------------------- Github user edwardcapriolo commented on a diff in the pull request: https://github.com/apache/incubator-gossip/pull/47#discussion_r112102487 --- Diff: gossip-base/src/main/java/org/apache/gossip/manager/GossipCore.java --- @@ -184,30 +138,23 @@ public void receive(Base base) { /** * Sends a blocking message. + * todo: move functionality to TransportManager layer. * @param message * @param uri * @throws RuntimeException if data can not be serialized or in transmission error */ - private void sendInternal(Base message, URI uri){ + private void sendInternal(Base message, URI uri) { byte[] json_bytes; try { - if (privKey == null){ - json_bytes = gossipManager.getObjectMapper().writeValueAsBytes(message); - } else { - SignedPayload p = new SignedPayload(); - p.setData(gossipManager.getObjectMapper().writeValueAsString(message).getBytes()); - p.setSignature(sign(p.getData())); - json_bytes = gossipManager.getObjectMapper().writeValueAsBytes(p); - } + // marshal the message. --- End diff -- We can avoid comments that seem clear > create gossip-transport-udp module > ---------------------------------- > > Key: GOSSIP-79 > URL: https://issues.apache.org/jira/browse/GOSSIP-79 > Project: Gossip > Issue Type: Improvement > Reporter: Gary Dusbabek > Assignee: Gary Dusbabek > > Create a transport module that houses the UDP transport. > This will probably require some refactoring. It may be prudent to create a > few interfaces that are kept in gossip-core which are then implemented in > gossip-transport-udp. > This probably needs to be a modules > 1 udp-server > 2 udp-client > OtherThings: > GossipManager.build() needs to look at the URI and dynamically load the right > server > GossipCore.send() needs to locate the right client for a URI. > Both of these things probably need a registry. Please dont make it a static > singleton thing ! :) -- This message was sent by Atlassian JIRA (v6.3.15#6346)