patricklucas opened a new pull request, #13673:
URL: https://github.com/apache/iceberg/pull/13673

   The `filterAndRemovePrefix` method is not used elsewhere and was already 
fairly specific to this use-case given it returns `Properties` to pass to a 
JDBC driver rather than operating generally on a `Map`.
   
   This change renames the method to `prepareJdbcDriverProperties` and adds the 
additional behavior of filtering-out the three properties specific to 
JdbcCatalog.
   
   Closes #13672
   
   Could I get a second opinion on if any of these three properties might ever 
need to be passed down to the JDBC driver?


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to