This is an automated email from the ASF dual-hosted git repository.
lhotari pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/pulsar-client-reactive.git
The following commit(s) were added to refs/heads/main by this push:
new 17b7252 Add missing annotations in MutableReactiveMessageSenderSpec
(#20)
17b7252 is described below
commit 17b72529563652579c248f74b55d1b4b5ef56028
Author: Christophe Bornet <[email protected]>
AuthorDate: Thu Nov 17 19:23:53 2022 +0100
Add missing annotations in MutableReactiveMessageSenderSpec (#20)
Also remove useless initialSubscriptionName (this is internal to the client)
---
.../api/MutableReactiveMessageSenderSpec.java | 36 ++++++++++++++++------
.../client/api/ReactiveMessageSenderBuilder.java | 5 ---
2 files changed, 26 insertions(+), 15 deletions(-)
diff --git
a/pulsar-client-reactive-api/src/main/java/org/apache/pulsar/reactive/client/api/MutableReactiveMessageSenderSpec.java
b/pulsar-client-reactive-api/src/main/java/org/apache/pulsar/reactive/client/api/MutableReactiveMessageSenderSpec.java
index 7d6bda7..9c0ccd5 100644
---
a/pulsar-client-reactive-api/src/main/java/org/apache/pulsar/reactive/client/api/MutableReactiveMessageSenderSpec.java
+++
b/pulsar-client-reactive-api/src/main/java/org/apache/pulsar/reactive/client/api/MutableReactiveMessageSenderSpec.java
@@ -86,8 +86,6 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
private Map<String, String> properties;
- private String initialSubscriptionName;
-
public MutableReactiveMessageSenderSpec() {
}
@@ -125,6 +123,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
? Collections.unmodifiableMap(new
LinkedHashMap<>(senderSpec.getProperties())) : null;
}
+ @Override
public String getTopicName() {
return this.topicName;
}
@@ -133,6 +132,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.topicName = topicName;
}
+ @Override
public String getProducerName() {
return this.producerName;
}
@@ -141,6 +141,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.producerName = producerName;
}
+ @Override
public Duration getSendTimeout() {
return this.sendTimeout;
}
@@ -149,6 +150,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.sendTimeout = sendTimeout;
}
+ @Override
public Integer getMaxPendingMessages() {
return this.maxPendingMessages;
}
@@ -157,6 +159,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.maxPendingMessages = maxPendingMessages;
}
+ @Override
public Integer getMaxPendingMessagesAcrossPartitions() {
return this.maxPendingMessagesAcrossPartitions;
}
@@ -165,6 +168,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.maxPendingMessagesAcrossPartitions =
maxPendingMessagesAcrossPartitions;
}
+ @Override
public MessageRoutingMode getMessageRoutingMode() {
return this.messageRoutingMode;
}
@@ -173,6 +177,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.messageRoutingMode = messageRoutingMode;
}
+ @Override
public HashingScheme getHashingScheme() {
return this.hashingScheme;
}
@@ -181,6 +186,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.hashingScheme = hashingScheme;
}
+ @Override
public ProducerCryptoFailureAction getCryptoFailureAction() {
return this.cryptoFailureAction;
}
@@ -189,6 +195,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.cryptoFailureAction = cryptoFailureAction;
}
+ @Override
public MessageRouter getMessageRouter() {
return this.messageRouter;
}
@@ -197,6 +204,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.messageRouter = messageRouter;
}
+ @Override
public Duration getBatchingMaxPublishDelay() {
return this.batchingMaxPublishDelay;
}
@@ -205,6 +213,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.batchingMaxPublishDelay = batchingMaxPublishDelay;
}
+ @Override
public Integer getRoundRobinRouterBatchingPartitionSwitchFrequency() {
return this.roundRobinRouterBatchingPartitionSwitchFrequency;
}
@@ -214,6 +223,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.roundRobinRouterBatchingPartitionSwitchFrequency =
roundRobinRouterBatchingPartitionSwitchFrequency;
}
+ @Override
public Integer getBatchingMaxMessages() {
return this.batchingMaxMessages;
}
@@ -222,6 +232,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.batchingMaxMessages = batchingMaxMessages;
}
+ @Override
public Integer getBatchingMaxBytes() {
return this.batchingMaxBytes;
}
@@ -230,6 +241,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.batchingMaxBytes = batchingMaxBytes;
}
+ @Override
public Boolean getBatchingEnabled() {
return this.batchingEnabled;
}
@@ -238,6 +250,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.batchingEnabled = batchingEnabled;
}
+ @Override
public BatcherBuilder getBatcherBuilder() {
return this.batcherBuilder;
}
@@ -246,6 +259,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.batcherBuilder = batcherBuilder;
}
+ @Override
public Boolean getChunkingEnabled() {
return this.chunkingEnabled;
}
@@ -254,6 +268,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.chunkingEnabled = chunkingEnabled;
}
+ @Override
public CryptoKeyReader getCryptoKeyReader() {
return this.cryptoKeyReader;
}
@@ -262,6 +277,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.cryptoKeyReader = cryptoKeyReader;
}
+ @Override
public Set<String> getEncryptionKeys() {
return this.encryptionKeys;
}
@@ -270,6 +286,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.encryptionKeys = encryptionKeys;
}
+ @Override
public CompressionType getCompressionType() {
return this.compressionType;
}
@@ -278,6 +295,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.compressionType = compressionType;
}
+ @Override
public Long getInitialSequenceId() {
return this.initialSequenceId;
}
@@ -286,6 +304,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.initialSequenceId = initialSequenceId;
}
+ @Override
public Boolean getAutoUpdatePartitions() {
return this.autoUpdatePartitions;
}
@@ -294,6 +313,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.autoUpdatePartitions = autoUpdatePartitions;
}
+ @Override
public Duration getAutoUpdatePartitionsInterval() {
return this.autoUpdatePartitionsInterval;
}
@@ -302,6 +322,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.autoUpdatePartitionsInterval =
autoUpdatePartitionsInterval;
}
+ @Override
public Boolean getMultiSchema() {
return this.multiSchema;
}
@@ -310,6 +331,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.multiSchema = multiSchema;
}
+ @Override
public ProducerAccessMode getAccessMode() {
return this.accessMode;
}
@@ -318,6 +340,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.accessMode = accessMode;
}
+ @Override
public Boolean getLazyStartPartitionedProducers() {
return this.lazyStartPartitionedProducers;
}
@@ -326,6 +349,7 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.lazyStartPartitionedProducers =
lazyStartPartitionedProducers;
}
+ @Override
public Map<String, String> getProperties() {
return this.properties;
}
@@ -334,14 +358,6 @@ public class MutableReactiveMessageSenderSpec implements
ReactiveMessageSenderSp
this.properties = properties;
}
- public String getInitialSubscriptionName() {
- return this.initialSubscriptionName;
- }
-
- public void setInitialSubscriptionName(String initialSubscriptionName) {
- this.initialSubscriptionName = initialSubscriptionName;
- }
-
public void applySpec(ReactiveMessageSenderSpec senderSpec) {
if (senderSpec.getTopicName() != null) {
setTopicName(senderSpec.getTopicName());
diff --git
a/pulsar-client-reactive-api/src/main/java/org/apache/pulsar/reactive/client/api/ReactiveMessageSenderBuilder.java
b/pulsar-client-reactive-api/src/main/java/org/apache/pulsar/reactive/client/api/ReactiveMessageSenderBuilder.java
index e3fc4b3..b4ddc52 100644
---
a/pulsar-client-reactive-api/src/main/java/org/apache/pulsar/reactive/client/api/ReactiveMessageSenderBuilder.java
+++
b/pulsar-client-reactive-api/src/main/java/org/apache/pulsar/reactive/client/api/ReactiveMessageSenderBuilder.java
@@ -178,11 +178,6 @@ public interface ReactiveMessageSenderBuilder<T> {
return this;
}
- default ReactiveMessageSenderBuilder<T> initialSubscriptionName(String
initialSubscriptionName) {
-
getMutableSpec().setInitialSubscriptionName(initialSubscriptionName);
- return this;
- }
-
ReactiveMessageSender<T> build();
}