[GitHub] [spark] maropu commented on a change in pull request #31384: [SPARK-31816][SQL] Added high level description about JDBC connection providers for users/developers

[GitBox <gi...@apache.org>]

maropu commented on a change in pull request #31384:
URL: https://github.com/apache/spark/pull/31384#discussion_r567824721



##########
File path: docs/sql-data-sources-jdbc.md
##########
@@ -202,13 +202,27 @@ the following case-insensitive options:
     <td><code>keytab</code></td>
     <td>
      Location of the kerberos keytab file (which must be pre-uploaded to all nodes either by <code>--files</code> option of spark-submit or manually) for the JDBC client. When path information found then Spark considers the keytab distributed manually, otherwise <code>--files</code> assumed. If both <code>keytab</code> and <code>principal</code> are defined then Spark tries to do kerberos authentication.
+     It worth to mention that kerberos authentication with keytab is not always supported by the JDBC driver.<br><br>
+     Before using this feature please make sure the following requirements met:
+      <ol>
+         <li> The included JDBC driver version supports kerberos authentication with keytab. </li>
+         <li> There is an embedded connection provider which supports the used database (many database support already added, but since it's changing please double check the list). </li> 
+      </ol>
+     If any other kind of custom authentication is needed <code>JdbcConnectionProvider</code> developer API is added which allows developers to implement it.

Review comment:
       Ah, ok. It seems we don't need to do so.




----------------------------------------------------------------
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.

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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org