Github user koeninger commented on the issue:

    https://github.com/apache/spark/pull/15102
  
    You should not be assuming 0 for a starting offset for partitions you've 
just learned about.  You should be asking the underlying driver consumer what 
its position is.  This is yet another argument in favor of keeping the 
implementations the same until they have to be different, because the existing 
implementation at least tries to do this correctly.
    
    My bigger concern is that it looks like you guys are continuing to hack in 
a particular direction, without addressing my points or answering whether 
you're willing to let me help work on this.
    
    Have you made up your mind?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to