HeartSaVioR commented on a change in pull request #22238: 
[SPARK-25245][DOCS][SS] Explain regarding limiting modification on 
"spark.sql.shuffle.partitions" for structured streaming
URL: https://github.com/apache/spark/pull/22238#discussion_r242006153
 
 

 ##########
 File path: docs/structured-streaming-programming-guide.md
 ##########
 @@ -2812,6 +2812,19 @@ See [Input Sources](#input-sources) and [Output 
Sinks](#output-sinks) sections f
 
 # Additional Information
 
+**Notes**
+
+- There're couple of configurations which are not modifiable once you run the 
query. If you really want to make changes for these configurations, you have to 
discard checkpoint and start a new query.
 
 Review comment:
   Reworded expression looks shorter and better. Will address.
   
   > I'm not sure how much we need to explain why they're not modifiable
   
   Yeah, I agree this is something everyone has different view. Someone could 
think the page is a guide so don't need to include implementation details. 
Other one (including me) could think the information is described nowhere so 
need to be included altogether, especially this is the major difference between 
batch query and streaming query. 
   
   No strong opinion (verbose vs less wondering on end users) so I can follow 
up the decision. If you think this is redundant please let me know so that I 
can address.
   
   > I think the example is redundant below.
   
   Will address.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to