FANNG1 commented on PR #5737:
URL: https://github.com/apache/gravitino/pull/5737#issuecomment-2533569286

   The main problem of the current implementation is passing Azure secret key 
not the token(token name is not correct) to the client side,  The 
`StorageSharedKeyCredential` key problems exists in the client side not the 
Iceberg REST server side, which means we could update the Iceberg version when 
testing. or update the Iceberg version in the server side is also ok but I 
prefer to do it in another PR


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