prmoore77 commented on code in PR #38461:
URL: https://github.com/apache/arrow/pull/38461#discussion_r1377801521


##########
docs/source/java/flight_sql_jdbc_driver.rst:
##########
@@ -114,6 +114,21 @@ parameters are:
      - null
      - When TLS is enabled, the password for the certificate store
 
+   * - tlsRootCerts
+     - null
+     - Path to PEM-encoded root certificates for TLS - use this as
+       an alternative to ``trustStore``
+
+   * - clientCertificate
+     - null
+     - Path to PEM-encoded client mTLS certificate when the Flight
+       SQL server requires client verification.
+
+   * - clientKey

Review Comment:
   Hello – it could be added, but the Java Flight client doesn’t yet support 
that to my knowledge – so I just wanted to catch up to that for now.  We can 
add it in a later PR, unless you feel strongly that it should be included here.



##########
docs/source/java/flight_sql_jdbc_driver.rst:
##########
@@ -114,6 +114,21 @@ parameters are:
      - null
      - When TLS is enabled, the password for the certificate store
 
+   * - tlsRootCerts
+     - null
+     - Path to PEM-encoded root certificates for TLS - use this as
+       an alternative to ``trustStore``
+
+   * - clientCertificate
+     - null
+     - Path to PEM-encoded client mTLS certificate when the Flight
+       SQL server requires client verification.
+
+   * - clientKey

Review Comment:
   Hello – it could be added, but the Java Flight client doesn’t yet support 
that to my knowledge – so I just wanted to catch up to that for now.  We can 
add it in a later PR, unless you feel strongly that it should be included here.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to