[GitHub] zeppelin issue #1594: [ZEPPELIN-1614] Remove markdown4j dep

[Leemoonsoo]

Github user Leemoonsoo commented on the issue:

    https://github.com/apache/zeppelin/pull/1594
  
    Thanks @1ambda for the contribution.
    I have compared output from markdown4j and pegdown and found some 
differences.
    
    font difference
    
![image](https://cloud.githubusercontent.com/assets/1540981/20041478/77b2d5be-a41e-11e6-8c62-e69308dcc87f.png)
    
    emphasis inside of `()`
    
![image](https://cloud.githubusercontent.com/assets/1540981/20041487/9730d67a-a41e-11e6-93eb-e103e767c08c.png)
    
    There could be more unknown differences. That means, existing notebook 
might render differently with pegdown and that could be unexpected result in 
some cases.
    
    So how about keep markdown4j and make pegdown default parser? That at least 
give user option to configure markdown interpreter with old markdown4j parser 
when they don't want to make change on existing notebook.


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