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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to