tillrohrmann commented on a change in pull request #9474:
[FLINK-13761][ScalaAPI]`SplitStream` should be deprecated because
`SplitJavaStream` is deprecated
URL: https://github.com/apache/flink/pull/9474#discussion_r316556530
##########
File path:
flink-streaming-scala/src/main/scala/org/apache/flink/streaming/api/scala/SplitStream.scala
##########
@@ -28,6 +28,7 @@ import org.apache.flink.streaming.api.datastream.{
SplitStream => SplitJavaStrea
* To apply a transformation on the whole output simply call
* the appropriate method on this stream.
*/
+@Deprecated
Review comment:
Scala's deprecated annotation is `@deprecated`.
Moreover, could we add a deprecation message explaining why it shouldn't be
used anymore? `@deprecated("reason", "deprecated since.")`
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services