sunxiaojian commented on code in PR #7734:
URL: https://github.com/apache/seatunnel/pull/7734#discussion_r1776212297


##########
seatunnel-connectors-v2/connector-iceberg/src/main/java/org/apache/seatunnel/connectors/seatunnel/iceberg/IcebergCatalogLoader.java:
##########
@@ -170,11 +169,10 @@ public static void doKerberosAuthentication(
                         "Start Kerberos authentication using principal {} and 
keytab {}",
                         principal,
                         keytabPath);
-                sun.security.krb5.Config.refresh();

Review Comment:
   > If your application is running and you modify the krb5.conf file at the 
same time. LoginUserFromKeytabAndReturnUGI will not reload the changes. It will 
only read and load the current Kerberos configuration when the application 
starts.
   > 
   > I think the correct way to modify it is to find out why this error is 
reported instead of directly removing it
   
   @dailai Please explain the impact of removing it



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