Sujit Madiraju
Sujit Madiraju
CC @majetideepak
@mbasmanova I've refactored `castFromDateString()` to use the enum directly. Updated docs and tests accordingly as well, please take a look.
@mbasmanova Thanks for all your feedback. Cleaned up and elaborated upon the docs, please let me if it looks good to you!
Came across this same issue, did you figure out any workaround?
Implementation has been completed. Opened a PR - https://github.com/facebookincubator/velox/pull/8823
@pedroerp no worries, thanks for all your feedback! Fixed the build errors, had to update my `initialize()` method - and added docs as I realized I'd forgotten that.