[GitHub] [spark] mridulm commented on pull request #42685: [WIP][SPARK-44937][CORE] Add SSL/TLS support for RPC and Shuffle communications

2023-09-26 Thread via GitHub
mridulm commented on PR #42685: URL: https://github.com/apache/spark/pull/42685#issuecomment-1736016000 > Just to confirm (since I'm not sure this repo has support for stacked PRs - if there is, please link me to an example) - you're proposing I put up one PR, get it approved and merged,

[GitHub] [spark] mridulm commented on pull request #42685: [WIP][SPARK-44937][CORE] Add SSL/TLS support for RPC and Shuffle communications

2023-09-26 Thread via GitHub
mridulm commented on PR #42685: URL: https://github.com/apache/spark/pull/42685#issuecomment-1736014865 +CC @otterc -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [spark] mridulm commented on pull request #42685: [WIP][SPARK-44937][CORE] Add SSL/TLS support for RPC and Shuffle communications

2023-09-22 Thread via GitHub
mridulm commented on PR #42685: URL: https://github.com/apache/spark/pull/42685#issuecomment-1732173622 Thanks for working on this @hasnain-db , this is a very nice adding to spark ! Given the size of the PR, can we split this up to make it easier to review ? -- This is an automated