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

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

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



##########
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:
       Kia ora @vvcephei I have removed the comment in my latest commit :)




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