felixcheung 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_r287584186
########## 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: should this be set to the default the same as interpreter.json ? ---------------------------------------------------------------- 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
