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


Reply via email to