Github user tzulitai commented on the issue:

    https://github.com/apache/flink/pull/2925
  
    Thanks for working on this @tony810430!
    
    I'd like to discuss a bit on the design before a more detailed review for 
this PR. If I'm correct, with this PR we will be creating a separate shard 
consuming thread with the Timer even if the fetch interval is 0 (which is our 
default). In this case, especially that it's our default, this extra thread 
usage seems sub-optimal to me, and I'm wondering if there's a better design for 
this. What do you think?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to