lordcheng10 commented on PR #17151:
URL: https://github.com/apache/pulsar/pull/17151#issuecomment-1228020723
> I think we can't resolve the issue("the subscription cannot be deleted")
entirely by this approach. Because we can't unsubscribe existing subscriptions,
also this approach can only affect REST API.
>
> Moreover, I think the current restriction is too strict for me. For
example, I want to use dots for subscription names. Could you modify this
restriction to be able to enable/disable it?
>
> ```
> % ./bin/pulsar-admin topics create-subscription -s api.sub
persistent://public/default/topic
> 2022-08-24T18:00:31,752+0900 [AsyncHttpClient-7-1] WARN
org.apache.pulsar.client.admin.internal.BaseResource -
[http://localhost:8080/admin/v2/persistent/public/default/topic/subscription/api.sub?replicated=false]
Failed to perform http put request: javax.ws.rs.BadRequestException: HTTP 400
subscriptionName contains special characters
> subscriptionName contains special characters
>
> Reason: subscriptionName contains special characters
> ```
IMO,
1. First, the subscription name should not contain special characters, and
the specific naming rules can be discussed;
2. For the existing illegal subscription name, it can be deleted by
dynamically opening the subscription name expiration cleanup, but it cannot be
deleted by command, because invisible characters (eg: /u0000) cannot be input
by command；
@equanz @codelipenghui @HQebupt @Technoboy- PTAL,thanks!
--
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]
