jongyoul commented on a change in pull request #4223:
URL: https://github.com/apache/zeppelin/pull/4223#discussion_r711175060



##########
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:
       It could be but `Pegdown` uses `asm` pages inside it but it's related to 
the implementation by Jetty. So without it, tests wouldn't be passed properly.

##########
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:
       It could be but `Pegdown` uses `asm` package inside it but it's related 
to the implementation by Jetty. So without it, tests wouldn't be passed 
properly.




-- 
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: dev-unsubscr...@zeppelin.apache.org

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


Reply via email to