Bastien Gysler

Hi Evan! Thanks for spending time looking into this issue. Yes, `extract_metadata` is indeed not that robust. However, both `readstat` and `extract_metadata` are saving us a lot of time in...

Thanks for being responsive! Currently, I'm the only developer working on this project, so I would be happy to try implementing this flag on readstat if you point me where...

@evanmiller As a side note related to my point on the PR. // All-or-nothing is probably not the best strategy for data type extraction. // When SPSS/STATA introduces new types,...

I definitely understand why/how this can happen and I don't try to minimise the problem or your efforts. Since you mentioned on the PR that you prefer this all-or-nothing design...

XML has a way to describe a null value, which is `xsi:nill`. e.g. `` In some cases, an empty string can be considered an absence of value, but not always....

Yes. As I mentioned, an empty string can be considered as an absence of value in some cases. For that reason, I am definitely in favour of having the option...

> ... if one needs this level of configuration, then perhaps handling the XML with an native API would be the best choice. Fair point. We can start with a...

Hi Gary, I am really sorry for the delay. I haven't had much time to dedicate to this project lately. Yes someone implemented this functionality on PR #16. I think...

I tested the patch and the basic setup looks good. However, the `tls` parameters are not supported. ```shell --set server.config.persistence.default.sql.tls.enabled=true \ --set server.config.persistence.default.sql.tls.enableHostVerification=true \ --set server.config.persistence.visibility.sql.tls.enabled=true \ --set server.config.persistence.visibility.sql.tls.enableHostVerification=true \...