You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by mm...@apache.org on 2019/06/14 00:10:32 UTC
[pulsar] branch master updated: Expose `Replicated From` info in
Message (#4524)
This is an automated email from the ASF dual-hosted git repository.
mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new f33be02 Expose `Replicated From` info in Message (#4524)
f33be02 is described below
commit f33be02aef139af5e03ebe955288d9e445575935
Author: Jia Zhai <zh...@apache.org>
AuthorDate: Fri Jun 14 08:10:27 2019 +0800
Expose `Replicated From` info in Message (#4524)
* expose replicated info in message
* add negative test case
---
.../java/org/apache/pulsar/client/api/Message.java | 17 +++++
.../org/apache/pulsar/client/impl/MessageImpl.java | 2 +
.../pulsar/client/impl/TopicMessageImpl.java | 10 +++
.../org/apache/pulsar/client/impl/MessageTest.java | 83 ++++++++++++++++++++++
4 files changed, 112 insertions(+)
diff --git a/pulsar-client-api/src/main/java/org/apache/pulsar/client/api/Message.java b/pulsar-client-api/src/main/java/org/apache/pulsar/client/api/Message.java
index e192916..6041632 100644
--- a/pulsar-client-api/src/main/java/org/apache/pulsar/client/api/Message.java
+++ b/pulsar-client-api/src/main/java/org/apache/pulsar/client/api/Message.java
@@ -202,4 +202,21 @@ public interface Message<T> {
* @return Schema version of the message if the message is produced with schema otherwise null.
*/
byte[] getSchemaVersion();
+
+ /**
+ * Check whether the message is replicated from other cluster.
+ *
+ * @since 2.4.0
+ * @return true if the message is replicated from other cluster.
+ * false otherwise.
+ */
+ boolean isReplicated();
+
+ /**
+ * Get name of cluster, from which the message is replicated.
+ *
+ * @since 2.4.0
+ * @return the name of cluster, from which the message is replicated.
+ */
+ String getReplicatedFrom();
}
diff --git a/pulsar-client/src/main/java/org/apache/pulsar/client/impl/MessageImpl.java b/pulsar-client/src/main/java/org/apache/pulsar/client/impl/MessageImpl.java
index 7fc37cc..fab8683 100644
--- a/pulsar-client/src/main/java/org/apache/pulsar/client/impl/MessageImpl.java
+++ b/pulsar-client/src/main/java/org/apache/pulsar/client/impl/MessageImpl.java
@@ -189,11 +189,13 @@ public class MessageImpl<T> implements Message<T> {
msgMetadataBuilder.setReplicatedFrom(cluster);
}
+ @Override
public boolean isReplicated() {
checkNotNull(msgMetadataBuilder);
return msgMetadataBuilder.hasReplicatedFrom();
}
+ @Override
public String getReplicatedFrom() {
checkNotNull(msgMetadataBuilder);
return msgMetadataBuilder.getReplicatedFrom();
diff --git a/pulsar-client/src/main/java/org/apache/pulsar/client/impl/TopicMessageImpl.java b/pulsar-client/src/main/java/org/apache/pulsar/client/impl/TopicMessageImpl.java
index 723875c..fb62ba3 100644
--- a/pulsar-client/src/main/java/org/apache/pulsar/client/impl/TopicMessageImpl.java
+++ b/pulsar-client/src/main/java/org/apache/pulsar/client/impl/TopicMessageImpl.java
@@ -158,6 +158,16 @@ public class TopicMessageImpl<T> implements Message<T> {
return msg.getSchemaVersion();
}
+ @Override
+ public boolean isReplicated() {
+ return msg.isReplicated();
+ }
+
+ @Override
+ public String getReplicatedFrom() {
+ return msg.getReplicatedFrom();
+ }
+
public Message<T> getMessage() {
return msg;
}
diff --git a/pulsar-client/src/test/java/org/apache/pulsar/client/impl/MessageTest.java b/pulsar-client/src/test/java/org/apache/pulsar/client/impl/MessageTest.java
new file mode 100644
index 0000000..4b96c0a
--- /dev/null
+++ b/pulsar-client/src/test/java/org/apache/pulsar/client/impl/MessageTest.java
@@ -0,0 +1,83 @@
+/**
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements. See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership. The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied. See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.pulsar.client.impl;
+
+import static org.testng.Assert.assertEquals;
+import static org.testng.Assert.assertFalse;
+import static org.testng.Assert.assertTrue;
+
+import java.nio.ByteBuffer;
+import org.apache.pulsar.client.api.Message;
+import org.apache.pulsar.client.api.Schema;
+import org.apache.pulsar.common.api.proto.PulsarApi.MessageMetadata;
+import org.testng.annotations.Test;
+
+/**
+ * Unit test of {@link Message} methods.
+ */
+public class MessageTest {
+
+ @Test
+ public void testMessageImplReplicatedInfo() {
+ String from = "ClusterNameOfReplicatedFrom";
+ MessageMetadata.Builder builder = MessageMetadata.newBuilder().setReplicatedFrom(from);
+ ByteBuffer payload = ByteBuffer.wrap(new byte[0]);
+ Message<byte[]> msg = MessageImpl.create(builder, payload, Schema.BYTES);
+
+ assertTrue(msg.isReplicated());
+ assertEquals(msg.getReplicatedFrom(), from);
+ }
+
+ @Test
+ public void testMessageImplNoReplicatedInfo() {
+ MessageMetadata.Builder builder = MessageMetadata.newBuilder();
+ ByteBuffer payload = ByteBuffer.wrap(new byte[0]);
+ Message<byte[]> msg = MessageImpl.create(builder, payload, Schema.BYTES);
+
+ assertFalse(msg.isReplicated());
+ assertTrue(msg.getReplicatedFrom().isEmpty());
+ }
+
+ @Test
+ public void testTopicMessageImplReplicatedInfo() {
+ String from = "ClusterNameOfReplicatedFromForTopicMessage";
+ String topicName = "myTopic";
+ MessageMetadata.Builder builder = MessageMetadata.newBuilder().setReplicatedFrom(from);
+ ByteBuffer payload = ByteBuffer.wrap(new byte[0]);
+ MessageImpl<byte[]> msg = MessageImpl.create(builder, payload, Schema.BYTES);
+ msg.setMessageId(new MessageIdImpl(-1, -1, -1));
+ TopicMessageImpl<byte[]> topicMessage = new TopicMessageImpl<>(topicName, topicName, msg);
+
+ assertTrue(topicMessage.isReplicated());
+ assertEquals(msg.getReplicatedFrom(), from);
+ }
+
+ @Test
+ public void testTopicMessageImplNoReplicatedInfo() {
+ String topicName = "myTopic";
+ MessageMetadata.Builder builder = MessageMetadata.newBuilder();
+ ByteBuffer payload = ByteBuffer.wrap(new byte[0]);
+ MessageImpl<byte[]> msg = MessageImpl.create(builder, payload, Schema.BYTES);
+ msg.setMessageId(new MessageIdImpl(-1, -1, -1));
+ TopicMessageImpl<byte[]> topicMessage = new TopicMessageImpl<>(topicName, topicName, msg);
+
+ assertFalse(topicMessage.isReplicated());
+ assertTrue(topicMessage.getReplicatedFrom().isEmpty());
+ }
+}