dawidwys commented on a change in pull request #13509:
URL: https://github.com/apache/flink/pull/13509#discussion_r497332354



##########
File path: 
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/TimeCharacteristic.java
##########
@@ -19,12 +19,26 @@
 package org.apache.flink.streaming.api;
 
 import org.apache.flink.annotation.PublicEvolving;
+import org.apache.flink.api.common.ExecutionConfig;
+import org.apache.flink.api.common.eventtime.WatermarkStrategy;
 
 /**
- * The time characteristic defines how the system determines time for 
time-dependent
- * order and operations that depend on time (such as time windows).
+ * The time characteristic defines how the system determines time for 
time-dependent order and
+ * operations that depend on time (such as time windows).
+ *
+ * @deprecated In Flink 1.12 the default stream time characteristic has been 
changed to {@link

Review comment:
       Sorry, my bad. Before this class there were python files and I somehow 
confused this with the python equivalent.
   
   BTW, do you think we should deprecate the `setStreamTimeCharacteristic` in 
python as well (or even remove it, as it was added in 1.12?)




----------------------------------------------------------------
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]


Reply via email to