Extend MongoDB connector support binData

[exxiang]

Description

Extend MongoDB connector support binData

Motivation and Context

Extend MongoDB connector support [binData] #23083

Impact

Test Plan

Contributor checklist

• Please make sure your submission complies with our development, formatting, commit message, and attribution guidelines.
• PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
• Documented new properties (with its default value), SQL syntax, functions, or other functionality.
• If release notes are required, they follow the release notes guidelines.
• Adequate tests were added if applicable.
• CI passed.

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== RELEASE NOTES ==

MongoDB Connector
* Support varbinary data type in MongoDB (pr:`23386`)

[exxiang]

@tdcmeehan I create a new PR replace #23084 , please review again, Thank!

[jaystarshot]

https://github.com/prestodb/presto/blob/master/presto-mongodb/src/test/java/com/facebook/presto/mongodb/TestMongoIntegrationSmokeTest.java#L102
I think extending these tests will be good

[exxiang]

https://github.com/prestodb/presto/blob/master/presto-mongodb/src/test/java/com/facebook/presto/mongodb/TestMongoIntegrationSmokeTest.java#L102 I think extending these tests will be good

presto/presto-mongodb/src/test/java/com/facebook/presto/mongodb/TestMongoIntegrationSmokeTest.java

Line 109 in 83a82cd

", vb varbinary" +

The varbinary TYPE test already exists

[steveburnett]

Is any update needed to https://github.com/prestodb/presto/blob/master/presto-docs/src/main/sphinx/connector/mongodb.rst for this PR?

[exxiang]

Is any update needed to https://github.com/prestodb/presto/blob/master/presto-docs/src/main/sphinx/connector/mongodb.rst for this PR?

I think not anything need update

[tdcmeehan]

Can you add a test in TestMongoIntegrationSmokeTest that proves the data can be read in a query?

[exxiang]

Can you add a test in TestMongoIntegrationSmokeTest that proves the data can be read in a query?

I have add it. Please review again. Thank u

[tdcmeehan]

Looks good @exxiang , please squash commits. Thank you!

[exxiang]

Looks good @exxiang , please squash commits. Thank you!

Please tell me if my squash is OK?

[tdcmeehan]

@exxiang there is still a merge commit. https://stackoverflow.com/questions/5189560/how-do-i-squash-my-last-n-commits-together

[exxiang]

@exxiang there is still a merge commit. https://stackoverflow.com/questions/5189560/how-do-i-squash-my-last-n-commits-together

Please review again. Thank U! Thank U! Thank U!

[tdcmeehan]

@exxiang can you please add a release note? It seems we're adding new data type support, which is relevant.

[exxiang]

@exxiang can you please add a release note? It seems we're adding new data type support, which is relevant.

sure I has already add it