zentol commented on code in PR #23845:
URL: https://github.com/apache/flink/pull/23845#discussion_r1418790359


##########
docs/content/docs/deployment/config.md:
##########
@@ -289,6 +289,15 @@ Enabling RocksDB's native metrics may cause degraded 
performance and should be s
 ----
 ----
 
+# Traces

Review Comment:
   I think this is intentional such that the main heading isn't part of the ToC.
   
   > we shouldn't have more than a single Header lvl 1
   
   correct, this is probably also why the Configuration ToC is so crap.
   
   > I'm +1 for setting the startLevel = 1
   
   Can you just try it out and see how it looks?
   It would be good to be able to go to the main heading via the ToC afaict. 🤔 



-- 
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: issues-unsubscr...@flink.apache.org

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

Reply via email to