[GitHub] [spark] viirya commented on pull request #33401: [SPARK-35785][SS][FOLLOWUP] Remove ignored test from RocksDBSuite

2021-07-17 Thread GitBox
viirya commented on pull request #33401: URL: https://github.com/apache/spark/pull/33401#issuecomment-881956895 Thanks @dongjoon-hyun -- 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

[GitHub] [spark] viirya commented on pull request #33401: [SPARK-35785][SS][FOLLOWUP] Remove ignored test from RocksDBSuite

2021-07-17 Thread GitBox
viirya commented on pull request #33401: URL: https://github.com/apache/spark/pull/33401#issuecomment-881856613 @dongjoon-hyun Yea, thanks. Actually I should move the "we ..." out of the description. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [spark] viirya commented on pull request #33401: [SPARK-35785][SS][FOLLOWUP] Remove ignored test from RocksDBSuite

2021-07-16 Thread GitBox
viirya commented on pull request #33401: URL: https://github.com/apache/spark/pull/33401#issuecomment-881741226 cc @HeartSaVioR -- 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

[GitHub] [spark] viirya commented on pull request #33401: [SPARK-35785][SS][FOLLOWUP] Remove ignored test from RocksDBSuite

2021-07-16 Thread GitBox
viirya commented on pull request #33401: URL: https://github.com/apache/spark/pull/33401#issuecomment-881729466 cc @xuanyuanking If we don't need the test, I think we can just remove it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on