Github user vesense commented on the issue:

    https://github.com/apache/storm/pull/1739
  
    @HeartSaVioR 
    >There might be some ways to pass partition count to upstream, and easy way 
to do might be adding method to DataSourcesProvider. What I'm considering is 
that we're now only thinking about partition count, but Calcite supports table 
statistics which contains estimated row count (not available for streaming 
env., partitioning attributes, etc.). Is passing partition count exhaustive? 
I'm not sure.
    
    Perhaps I can give you some suggestions when I have a deeper understanding 
in Calcite. But now I think we can address STORM-2147 using a simple way before 
we get a better one.
    
    >As this is on top of STORM-1446 which needs understanding of Calcite, 
learning Calcite is more important for you to work on further works for Storm 
SQL, especially Storm SQL lacks reviewers to go forward. So if you haven't had 
time, please take your time to get it.
    
    Thanks for your advice, yes, learning Calcite is in my plan.


---
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