Bookkeeper
  1. Bookkeeper
  2. BOOKKEEPER-149

Implement HedwigSubscriber#getSubscriptionList(ByteString subscriberId)

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      This method currently returns null. (see https://github.com/apache/bookkeeper/blob/trunk/hedwig-client/src/main/java/org/apache/hedwig/client/netty/HedwigSubscriber.java#L436 )

      A possible implementation would use a control message, like subscription messages. But that will require modifications to the protocol definition. Maybe there is another way?

        Activity

        Matthieu Morel created issue -
        Hide
        Ivan Kelly added a comment -

        getSubscriptionList is only part of HedwigSubscriber. It's not part of org.apache.hedwig.client.api.Subscriber, so it would have to be added to that also.

        This sounds more like an admin method than a client method. What is the usecase for this?

        Show
        Ivan Kelly added a comment - getSubscriptionList is only part of HedwigSubscriber. It's not part of org.apache.hedwig.client.api.Subscriber, so it would have to be added to that also. This sounds more like an admin method than a client method. What is the usecase for this?
        Hide
        Matthieu Morel added a comment -

        Use case:

        Show
        Matthieu Morel added a comment - Use case: cleaning up topic subscriptions, more specifically for implementing JMS API's http://docs.oracle.com/javaee/6/api/javax/jms/TopicSession.html#unsubscribe(java.lang.String ) (in the case the client was disconnected, information about subscribed topic(s) may be lost, and getSubscriptionList is a way to retrieve this information)
        Hide
        Ivan Kelly added a comment -

        ah ok, this sounds like an admin thing then. I don't think you need to contact the hub to do this though. It should be possible to check this by going to zookeeper. Ben was talking before christmas, about doing an admin client. I think this would fit into that.

        Show
        Ivan Kelly added a comment - ah ok, this sounds like an admin thing then. I don't think you need to contact the hub to do this though. It should be possible to check this by going to zookeeper. Ben was talking before christmas, about doing an admin client. I think this would fit into that.

          People

          • Assignee:
            Unassigned
            Reporter:
            Matthieu Morel
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:

              Development