You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "ennuite (via GitHub)" <gi...@apache.org> on 2026/02/05 02:08:28 UTC

[PR] GH-1004: [JDBC] Fix NPE in ArrowFlightJdbcDriver#connect​(final String url, final Properties info) [arrow-java]

ennuite opened a new pull request, #1005:
URL: https://github.com/apache/arrow-java/pull/1005

   ## What's Changed
   `ArrowFlightJdbcDriver.connect(final String url, final Properties info)` now properly ignores a null value for `info`.
   It obtains the properties solely from the URL.
   
   Closes #1004.
   


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] GH-1004: [JDBC] Fix NPE in ArrowFlightJdbcDriver#connect​(final String url, final Properties info) [arrow-java]

Posted by "ennuite (via GitHub)" <gi...@apache.org>.
ennuite commented on PR #1005:
URL: https://github.com/apache/arrow-java/pull/1005#issuecomment-3942040423

   @jbonofre done. Is this ok to merge?


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] GH-1004: [JDBC] Fix NPE in ArrowFlightJdbcDriver#connect​(final String url, final Properties info) [arrow-java]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #1005:
URL: https://github.com/apache/arrow-java/pull/1005#issuecomment-3850673868

   
   <!-- label_helper_comment -->
   Thank you for opening a pull request!
   
   Please label the PR with one or more of:
   
   - bug-fix
   - chore
   - dependencies
   - documentation
   - enhancement
   
   Also, add the 'breaking-change' label if appropriate.
   
   See [CONTRIBUTING.md](https://github.com/apache/arrow-java/blob/main/CONTRIBUTING.md) for details.
   


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] GH-1004: [JDBC] Fix NPE in ArrowFlightJdbcDriver#connect​(final String url, final Properties info) [arrow-java]

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre merged PR #1005:
URL: https://github.com/apache/arrow-java/pull/1005


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] GH-1004: [JDBC] Fix NPE in ArrowFlightJdbcDriver#connect​(final String url, final Properties info) [arrow-java]

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #1005:
URL: https://github.com/apache/arrow-java/pull/1005#issuecomment-3867142750

   NB: the tests won't pass (see #1002). I will rebase as soon as CI is green again.


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] GH-1004: [JDBC] Fix NPE in ArrowFlightJdbcDriver#connect​(final String url, final Properties info) [arrow-java]

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #1005:
URL: https://github.com/apache/arrow-java/pull/1005#issuecomment-3914617603

   @ennuite can you please rebase this PR ? The CI should be ok now.


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org