codelipenghui commented on a change in pull request #14692:
URL: https://github.com/apache/pulsar/pull/14692#discussion_r826964565
##########
File path:
pulsar-client-admin-api/src/main/java/org/apache/pulsar/client/admin/Topics.java
##########
@@ -1595,7 +1596,52 @@ void expireMessagesForAllSubscriptions(String topic,
long expireTimeInSeconds)
* @throws PulsarAdminException
* Unexpected error
*/
- void createSubscription(String topic, String subscriptionName, MessageId
messageId)
+ @Deprecated
Review comment:
We don't need to deprecate this one?
##########
File path:
pulsar-client-admin-api/src/main/java/org/apache/pulsar/client/admin/Topics.java
##########
@@ -1595,7 +1596,52 @@ void expireMessagesForAllSubscriptions(String topic,
long expireTimeInSeconds)
* @throws PulsarAdminException
* Unexpected error
*/
- void createSubscription(String topic, String subscriptionName, MessageId
messageId)
+ @Deprecated
+ default void createSubscription(String topic, String subscriptionName,
MessageId messageId)
+ throws PulsarAdminException {
+ createSubscription(topic, subscriptionName, messageId, false);
+ };
+
+ /**
+ * Use {@link Topics#createSubscriptionAsync(String, String, MessageId,
boolean)} instead.
+ * Create a new subscription on a topic.
+ *
+ * @param topic
+ * topic name
+ * @param subscriptionName
+ * Subscription name
+ * @param messageId
+ * The {@link MessageId} on where to initialize the
subscription. It could be {@link MessageId#latest},
+ * {@link MessageId#earliest} or a specific message id.
+ */
+ @Deprecated
Review comment:
Same as the above comment.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
