[GitHub] [kafka] vvcephei commented on a change in pull request #8520: Add explicit grace period to tumbling window example

[GitBox <gi...@apache.org>]

vvcephei commented on a change in pull request #8520:
URL: https://github.com/apache/kafka/pull/8520#discussion_r411704050



##########
File path: docs/streams/developer-guide/dsl-api.html
##########
@@ -3262,12 +3262,15 @@ <h5><a class="toc-backref" href="#id16">KTable-KTable Foreign-Key
 <span class="kn">import</span> <span class="nn">org.apache.kafka.streams.kstream.TimeWindows</span><span class="o">;</span>
 
 <span class="c1">// A tumbling time window with a size of 5 minutes (and, by definition, an implicit</span>
-<span class="c1">// advance interval of 5 minutes).</span>
+<span class="c1">// advance interval of 5 minutes). Note the explicit grace period, as the current</span>
+<span class="c1">// default value is 24 hours, which may be larger than needed for smaller windows. </span>
+<span class="c1">// Note that this default may change in future major version releases.</span>

Review comment:
       I'd remove the note about changing the default out. We can either find a non-breaking way to change defaults (see https://issues.apache.org/jira/browse/KAFKA-8924?focusedCommentId=17088091&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17088091 ), or we can't change it.
   
   I'd rather not make people paranoid that we're going to break their app semantics from underneath them, because we won't.




----------------------------------------------------------------
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:
users@infra.apache.org