[GitHub] [pulsar] Anonymitaet commented on a diff in pull request #17100: [fix][doc] Fix links to broker configuration on Admin API Topics page

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

Anonymitaet commented on code in PR #17100:
URL: https://github.com/apache/pulsar/pull/17100#discussion_r951244807


##########
site2/docs/admin-api-topics.md:
##########
@@ -1666,7 +1666,7 @@ Non-partitioned topics must be explicitly created. When creating a new non-parti
 
 By default, 60 seconds after creation, topics are considered inactive and deleted automatically to avoid generating trash data. To disable this feature, set `brokerDeleteInactiveTopicsEnabled` to `false`. To change the frequency of checking inactive topics, set `brokerDeleteInactiveTopicsFrequencySeconds` to a specific value.
 
-For more information about the two parameters, see [here](reference-configuration.md#broker).
+For more information about the two parameters, see [here](reference-configuration-broker.md).

Review Comment:
   Hi @alpreu we're generating configuration docs from code automatically, and the new page will be something like https://pulsar.apache.org/tools/pulsar-config/2.11.0-SNAPSHOT/#/
   
   `reference-configuration-broker.md` will be removed from the pulsar repo https://github.com/apache/pulsar/pull/17197
   
   So can you hold on this change?
   
   Feel free to update it after the new page is finalized, thank you!
   
   /cc @SignorMercurio 



-- 
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: commits-unsubscribe@pulsar.apache.org

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