[GitHub] zeppelin issue #1384: [ZEPPELIN-1387] Support table syntax in markdown inter...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

    https://github.com/apache/zeppelin/pull/1384
  
    @1ambda Great! It works well as expected 👍 
    
    If you want to add the interpreter property(`markdown.parser.type`) to the 
markdown interpreter, it would be better to provide that information via 
[Zeppelin markdown interpreter 
docs](https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/interpreter/markdown.html)
 with configuration table as like the other interpreters do. Nothing much 
needed. You can just add the below conf table to `markdown.md`.
    ```
    ## Configuration
    <table class="table-configuration">
      <tr>
        <th>Name</th>
        <th>Default Value</th>
        <th>Description</th>
      </tr>
      <tr>
        <td>markdown.parser.type</td>
        <td>markdown4j </td>
        <td>Markdown Parser Type. <br/> Available values: markdown4j, 
pegdown.</td>
      </tr>
    </table>
    ```
    
    Also the below sentences in [Overview 
section](https://zeppelin.apache.org/docs/0.7.0-SNAPSHOT/interpreter/markdown.html#overview)
 need to be updated accordingly. 
    >Markdown is a plain text formatting syntax designed so that it can be 
converted to HTML. Apache Zeppelin uses markdown4j. For more examples and 
extension support, please checkout here.
    
    FYI, `markdown.md` file is under `ZEPPELIN_HOME/docs/interpreter/` :)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---