RyanSkraba commented on code in PR #23006:
URL: https://github.com/apache/flink/pull/23006#discussion_r1270790972
##########
flink-scala/src/main/java/org/apache/flink/api/scala/typeutils/ScalaOptionSerializerSnapshot.java:
##########
@@ -24,7 +24,17 @@
import scala.Option;
-/** A {@link TypeSerializerSnapshot} for the Scala {@link OptionSerializer}. */
+/**
+ * A {@link TypeSerializerSnapshot} for the Scala {@link OptionSerializer}.
+ *
+ * @deprecated All Flink Scala APIs are deprecated and will be removed in a
future Flink major
+ * version. You can still build your application in Scala, but you should
move to the Java
+ * version of either the DataStream and/or Table API.
+ * @see <a
+ *
href="https://cwiki.apache.org/confluence/display/FLINK/FLIP-265+Deprecate+and+remove+Scala+API+support";>
+ * FLIP-265 Deprecate and remove Scala API support</a>
+ */
+@Deprecated
public final class ScalaOptionSerializerSnapshot<E>
Review Comment:
Yeah, I think they all should have been `@Internal` -- my reasoning was that
removing a `@deprecated` annotation should probably always safe, even between
minor releases. Would you prefer these to be removed?
--
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]
