jongyoul commented on a change in pull request #4223:
URL: https://github.com/apache/zeppelin/pull/4223#discussion_r711986085
##########
File path:
zeppelin-jupyter/src/main/java/org/apache/zeppelin/jupyter/JupyterUtil.java
##########
@@ -78,7 +78,7 @@ public JupyterUtil() {
.registerSubtype(ExecuteResult.class, "execute_result")
.registerSubtype(DisplayData.class,
"display_data").registerSubtype(Stream.class, "stream")
.registerSubtype(Error.class, "error");
- this.markdownProcessor = new PegdownParser();
+ this.markdownProcessor = new FlexmarkParser();
Review comment:
Yeap. I think it should be handled as well. By the way, it looks like
having redundant dependencies on `zeppelin-jupyter` by having `markdown`.
`JupyterUtil` parser markdown paragraph but I don't think we don't have to
depend on the whole `markdown` module for that case. I think we'd better
decouple them and copy a small piece of code. #4228
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
