thswlsqls opened a new issue, #17047:
URL: https://github.com/apache/iceberg/issues/17047

   **Apache Iceberg version**
   main @ 035fc1e40
   
   **Query engine**
   N/A — engine-agnostic (aws/ client factory)
   
   **Please describe the bug**
   `AssumeRoleAwsClientFactory.glue()`/`.kms()` 
(`aws/src/main/java/org/apache/iceberg/aws/AssumeRoleAwsClientFactory.java`, 
lines 79-95) build clients without applying the `glue.endpoint`/`kms.endpoint` 
overrides configured via `AwsProperties` — they never call 
`applyGlueEndpointConfigurations`/`applyKmsEndpointConfigurations`. The sibling 
`dynamo()` in the same class correctly calls 
`.applyMutation(awsProperties::applyDynamoDbEndpointConfigurations)`, and 
`AwsClientFactories.DefaultAwsClientFactory.glue()`/`.kms()` 
(`aws/src/main/java/org/apache/iceberg/aws/AwsClientFactories.java`, lines 
145-164) both apply their respective endpoint configurations.
   
   `LakeFormationAwsClientFactory.kms()` 
(`aws/src/main/java/org/apache/iceberg/aws/lakeformation/LakeFormationAwsClientFactory.java`,
 lines 95-108, registered-table branch) has the same gap.
   
   **Steps to reproduce**
   Configure a catalog with 
`client.factory=org.apache.iceberg.aws.AssumeRoleAwsClientFactory`, plus 
`client.assume-role.arn`, `client.assume-role.region`, and 
`glue.endpoint`/`kms.endpoint` pointing at a custom endpoint (e.g. VPC or FIPS 
endpoint). Expected: `glue()`/`kms()` clients connect to the configured 
endpoint. Actual: the setting is silently ignored and the client connects to 
the default AWS endpoint.
   
   **Additional context**
   N/A — root cause and fix identified; PR forthcoming.
   


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