Sayari Mukherjee
Sayari Mukherjee
> Does https://github.com/prestodb/presto/blob/master/presto-docs/src/main/sphinx/connector/mongodb.rst need any documentation update for this? @steveburnett , yes the documentation will need to be updated. I'm currently working on addressing the review feedback from @agrawalreetika and...
> @imsayari404 There are compilation failures, please fix those, add the documentation & squash all your commits into one. @agrawalreetika The updated changes are now pushed to the branch.
> LGTM! (docs) > > Pull updated branch, new local doc build, looks good. Thank you! Appreciate your time and review, @steveburnett, thanks a lot for confirming the doc updates!
> Couple of high level comments - > > 1. Please delete SslContextProvider file from presto-cassandra module since it is no longer getting used. > 2. Please change commit message...
> Please check why test other modules check is failing. Thanks. Thank you @pratyakshsharma, sure.
@tdcmeehan, could you please review this PR at your convenience?
Thanks @tdcmeehan , I've squashed the commits @imjalpreet, could you please review this PR at your convenience?
I've pushed some changes related to the upgrade work, but I'm currently facing an issue while testing the Pinot connector locally on Presto OSS master. The error I'm seeing is:...