Github user yashs360 commented on the issue:

    https://github.com/apache/spark/pull/21541
  
    No this code path only gets invoked when the resume mode is specified as
    AT_TIMESTAMP. And it later usus Amazon's client which fails now. Having
    this change would not set the timestamp directly but via a different method
    which sets the timestamp for us and the resume mode as AT_TIMESTAMP.
    
    On Fri., 6 Jul. 2018, 11:43 pm Sean Owen, <[email protected]> wrote:
    
    > I don't know this code myself, but the change looks reasonable. A test is
    > ideal, but some failure modes are hard to test. However isn't this code
    > path always invoked when this receiver is used? would it not have failed
    > commonly already? But again I don't know this code.
    >
    > —
    > You are receiving this because you authored the thread.
    > Reply to this email directly, view it on GitHub
    > <https://github.com/apache/spark/pull/21541#issuecomment-403037972>, or 
mute
    > the thread
    > 
<https://github.com/notifications/unsubscribe-auth/Ag5-M8BzYkFoT8LvsXRhVFEIWQAcEozNks5uD2lvgaJpZM4Uk3Hq>
    > .
    >



---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to