yadavay-amzn commented on PR #57655: URL: https://github.com/apache/spark/pull/57655#issuecomment-5258690794
Thanks @dongjoon-hyun ! For the `VisibleForTesting` tag, this was requested by @sarutak in a previous round of review, so I want to double check before deciding how to proceed. We can either drop the tag/annotation and leave a plain comment (like `// visible for testing`) to avoid the dependency addition or keep the tag and add the guava dependency. -- 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 unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
