codelipenghui commented on a change in pull request #9225:
URL: https://github.com/apache/pulsar/pull/9225#discussion_r560057909
##########
File path:
pulsar-client-admin/src/main/java/org/apache/pulsar/client/admin/Topics.java
##########
@@ -1604,7 +1604,15 @@ void setDelayedDeliveryPolicy(String topic
* @throws PulsarAdminException
* Unexpected error
*/
- int getMessageTTL(String topic) throws PulsarAdminException;
+ Integer getMessageTTL(String topic) throws PulsarAdminException;
Review comment:
@eolivelli This method is introduced in 2.7.0(topic level policy) and I
think this is a mistake in 2.7.0, most of the `get policy` method are returned
Integer or Long, and the old behavior is the wrong behavior. I think we can
clarify it in the release doc, I prefer to fix it directly. If left this method
for future releases, users might be confused more.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
