liuxunorg commented on a change in pull request #3371: [ZEPPELIN-4171] Configuration paragraph is empty to execute URL: https://github.com/apache/zeppelin/pull/3371#discussion_r287587658
########## File path: zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Paragraph.java ########## @@ -105,10 +105,12 @@ private transient Map<String, String> localProperties = new HashMap<>(); private transient Map<String, ParagraphRuntimeInfo> runtimeInfos = new HashMap<>(); - private static String PARAGRAPH_CONFIG_RUNONSELECTIONCHANGE = "runOnSelectionChange"; + public static String PARAGRAPH_CONFIG_RUNONSELECTIONCHANGE = "runOnSelectionChange"; private static boolean PARAGRAPH_CONFIG_RUNONSELECTIONCHANGE_DEFAULT = true; - private static String PARAGRAPH_CONFIG_TITLE = "title"; + public static String PARAGRAPH_CONFIG_TITLE = "title"; private static boolean PARAGRAPH_CONFIG_TITLE_DEFAULT = false; + public static String PARAGRAPH_CONFIG_CHECK_EMTPY = "checkEmpty"; + private static boolean PARAGRAPH_CONFIG_CHECK_EMTPY_DEFAULT = true; Review comment: The default value is `checkEmpty=true`, Is consistent with the behavior of zeppelin now, If the content of the paragraph is empty, Can't execute. Only when `checkEmpty=false`, The paragraph is empty and will be executed. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
