vdiravka commented on pull request #2364: URL: https://github.com/apache/drill/pull/2364#issuecomment-985401139
> Sorry, I perhaps read the diffs backward before: the things I thought were missing seem actually fine. > > Just to double-check, there were some additional complex type handling and tests in [this commit](https://github.com/paul-rogers/drill/commit/7eefe01a623ade4ad387073686108ab658ce4322) that would be useful to add. Hi Paul. I double checked and looks like the changes from commit is already incorporated. I made cheery-pick, resolved 3 merge conflicts and after that there are no new changes. The other question what do you think is that useful to have dfs plugin config to switch to V1 JSON reader? I think, since the goal to switch to V2 fully, the system/session option is enough -- 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. To unsubscribe, e-mail: dev-unsubscr...@drill.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
