[ 
https://issues.apache.org/jira/browse/KAFKA-8633?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16881110#comment-16881110
 ] 

Weichu Liu commented on KAFKA-8633:
-----------------------------------

Seems no one is responding, so I will make a PR to fix this.

> Extra </td> in generated documents
> ----------------------------------
>
>                 Key: KAFKA-8633
>                 URL: https://issues.apache.org/jira/browse/KAFKA-8633
>             Project: Kafka
>          Issue Type: Task
>          Components: documentation
>            Reporter: Weichu Liu
>            Priority: Trivial
>
> The auto generated tables for all configurations (e.g. 
> https://kafka.apache.org/documentation/#brokerconfigs) are with 2 </td> for 
> each cell.
> e.g. the first row for broker configuration.
> {noformat}
> <tr>
> <td>zookeeper.connect</td></td><td>Specifies the ZooKeeper connection string 
> in the form <code>hostname:port</code> where host and port are the host and 
> port of a ZooKeeper server. To allow connecting through other ZooKeeper nodes 
> when that ZooKeeper machine is down you can also specify multiple hosts in 
> the form <code>hostname1:port1,hostname2:port2,hostname3:port3</code>.
> The server can also have a ZooKeeper chroot path as part of its ZooKeeper 
> connection string which puts its data under some path in the global ZooKeeper 
> namespace. For example to give a chroot path of <code>/chroot/path</code> you 
> would give the connection string as 
> <code>hostname1:port1,hostname2:port2,hostname3:port3/chroot/path</code>.</td></td><td>string</td></td><td></td></td><td></td></td><td>high</td></td><td>read-only</td></tr>
> {noformat}
> This is due to {{toHtmlTable}} function in 
> {{./clients/src/main/java/org/apache/kafka/common/config/ConfigDef.java}} is 
> appending an extra "</td>" in the code.
> {code:java}
>             for (String headerName : headers()) {
>                 addColumnValue(b, getConfigValue(key, headerName));
>                 b.append("</td>");
>             }
> {code}
> (The addColumnValue already wrap the value with <td> and </td>)
> This is very minor issue, but it will prevent an html parser to properly 
> fetch table data (like what I was trying to do)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to