panbingkun commented on code in PR #46019:
URL: https://github.com/apache/spark/pull/46019#discussion_r1562211489
##########
core/src/main/scala/org/apache/spark/api/python/WriteInputFormatTestDataGenerator.scala:
##########
@@ -104,6 +105,7 @@ private[python] class WritableToDoubleArrayConverter
extends Converter[Any, Arra
* This object contains method to generate SequenceFile test data and write it
to a
* given directory (probably a temp directory)
*/
+@deprecated("This class will be move to `test`.", "3.5.2")
Review Comment:
It seems to be only used in `Python related testing`, but it is indeed in
the folder `main` instead of the folder `test`. Can we move it directly without
doing any pre-deprecated operations in version `3.5`?
I'm not quite sure about this rule.
--
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]