You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "leesf (via GitHub)" <gi...@apache.org> on 2023/08/21 11:21:39 UTC

[GitHub] [spark] leesf commented on pull request #42553: [SPARK-44864] Align streaming statistics link format with other page links

leesf commented on PR #42553:
URL: https://github.com/apache/spark/pull/42553#issuecomment-1686142418

   > TBH `%s/%s/statistics?id=%s` is more "restful". (And, of course it would be totally restful if it is `%s/%s/statistics/%s`, but we can't make such changes.)
   
   @gengliangwang Thanks for the comment, so for now, do we need to change other page link format to make it more `restful` or align the streaming link with other page link for first stage unify?


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

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org