Github user adityakishore commented on the pull request:
https://github.com/apache/drill/pull/418#issuecomment-194097775
Until I looked at the code, I was under assumption that we are using
Jackson to extract the serializable properties. We can, and should, definitely
go that route.
The way code currently works is that it iterate through allthe table
options and see if there is a Java field present in the corresponding
FormatPluginConfig class. If it does find one, and this is why I say it is a
bug in the current implementation, it makes is accessible
(`setAccesible(true)`) implying that it is expected to work with non-public
fields and sets the value to the one passed as parameter.
---
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.
---
