[GitHub] spark issue #18029: [SPARK-20168] [DStream] Add changes to use kinesis fetch...

Github user yssharma commented on the issue: https://github.com/apache/spark/pull/18029 Hi @srowen we've iterated this patch to bring it in a good state. Need a committer ✔️ before we can go ahead merging it. --- If your project is set up for it, you can reply to this email

[GitHub] spark issue #18029: [SPARK-20168] [DStream] Add changes to use kinesis fetch...

Github user yssharma commented on the issue: https://github.com/apache/spark/pull/18029 Resolved conflict introduced by other code commits. --- 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

[GitHub] spark issue #18029: [SPARK-20168] [DStream] Add changes to use kinesis fetch...

Github user yssharma commented on the issue: https://github.com/apache/spark/pull/18029 Could I get some love here from the committers please @brkyvz @HyukjinKwon @srowen . Would love to work on any changes if required. --- If your project is set up for it, you can reply

[GitHub] spark issue #18029: [SPARK-20168] [DStream] Add changes to use kinesis fetch...

Github user yssharma commented on the issue: https://github.com/apache/spark/pull/18029 Thanks for the review @brkyvz . Please have a look at the new PR. I have implemented the review comments. For the API hygiene, I have added both the new and old API, and have marked the old

[GitHub] spark issue #19427: Reset spark.driver.bindAddress when starting a Checkpoin...

Github user yssharma commented on the issue: https://github.com/apache/spark/pull/19427 @ssaavedra Could you also update the Title as [SPARK-X][component] Title... please. --- - To unsubscribe, e-mail: reviews

[GitHub] spark issue #18029: [SPARK-20168] [DStream] Add changes to use kinesis fetch...

Github user yssharma commented on the issue: https://github.com/apache/spark/pull/18029 @brkyvz could you please have a look if it looks good. Would be great if you're happy with the changes and we could merge

[GitHub] spark issue #18029: [SPARK-20168] [DStream] Add changes to use kinesis fetch...

Github user yssharma commented on the issue: https://github.com/apache/spark/pull/18029 Thanks for being so helpful and patient on this one @brkyvz . I will leave this with you now for your final ☑️ if you're happy

[GitHub] spark issue #18029: [SPARK-20168] [DStream] Add changes to use kinesis fetch...

Github user yssharma commented on the issue: https://github.com/apache/spark/pull/18029 @brkyvz Please have a look once you have time. Thanks. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

[GitHub] spark issue #18029: [SPARK-20168] [DStream] Add changes to use kinesis fetch...

Github user yssharma commented on the issue: https://github.com/apache/spark/pull/18029 @brkyvz Squashed multiple commits into one for better readability. Please have a look when you get time. Thanks

<    1   2