Github user arina-ielchiieva commented on the issue:

    https://github.com/apache/drill/pull/655
  
    Answering to all comments in one:
    1. `${option.getValue()}` is not sufficient since method returns Object and 
freemarker does not know how to cast it.
    2. `?c` only works with boolean and numbers, since options can be string, 
it not sufficient
    3. `?string` works with most of the types but it's true for boolean seems 
to be deprecated
    4. to set null value to option is not possible (handled in 
SetOptionHandler.class) but option.getValue() may return null according to the 
code.
    
    Paul's idea to prepare options to display on Java side seems to be the most 
reasonable.
    So I have updated the pull request, added new method in 
ProfileWrapper.class with transforms `OptionValueList` into `Map<String, 
String>`. If option value is null, it will be displayed as word 'null'.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to