Skip to content

Conversation

@malhotrashivam
Copy link
Contributor

@github-actions
Copy link
Contributor

github-actions bot commented Sep 11, 2025

No docs changes detected for 9d09991

// actually calls it?), but we need to be able to compile against it to implement AwsClientFactory
compileOnly libs.awssdk.kms
// Needed for AwsClientFactory
implementation libs.awssdk.kms
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needed for new KMS related properties added in apache/iceberg#13136

@malhotrashivam malhotrashivam changed the title Upgraded Iceberg 1.9.2 -> 1.10.0 feat: Upgraded Iceberg 1.9.2 -> 1.10.0 Sep 11, 2025
@malhotrashivam malhotrashivam changed the title feat: Upgraded Iceberg 1.9.2 -> 1.10.0 feat: Upgraded Iceberg version 1.9.2 -> 1.10.0 Sep 11, 2025
}
}

@Test
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

apache/iceberg#12887 has been released.

.add("PositionDeleteFileCount", int.class, PartitionStats::positionDeleteFileCount)
.add("EqualityDeleteRecordCount", long.class, PartitionStats::equalityDeleteRecordCount)
.add("EqualityDeleteFileCount", int.class, PartitionStats::equalityDeleteFileCount)
.add("TotalRecordCount", long.class, PartitionStats::totalRecordCount)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awssdk-netty-nio = { module = "software.amazon.awssdk:netty-nio-client" }

# TODO (DH-19508) : Remove this dependency when https://github.com/apache/iceberg/issues/13133 fix is released
analyticsaccelerator-s3 = { module = "software.amazon.s3.analyticsaccelerator:analyticsaccelerator-s3", version.ref = "analyticsaccelerator" }
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

apache/iceberg#13134 has been released.

testImplementation TestTools.projectDependency(project, 'extensions-s3')
testImplementation TestTools.projectDependency(project, 'extensions-iceberg')

// TODO (DH-19508) : Remove this dependency when https://github.com/apache/iceberg/issues/13133 fix is released
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

apache/iceberg#13134 has been released.

@malhotrashivam malhotrashivam changed the title feat: Upgraded Iceberg version 1.9.2 -> 1.10.0 feat: Bump Iceberg version from 1.9.2 to 1.10.0 Sep 11, 2025
@malhotrashivam malhotrashivam changed the title feat: Bump Iceberg version from 1.9.2 to 1.10.0 feat: bump org.apache.iceberg:iceberg-bom from 1.9.2 to 1.10.0 Sep 11, 2025
@malhotrashivam malhotrashivam changed the title feat: bump org.apache.iceberg:iceberg-bom from 1.9.2 to 1.10.0 chore: bump org.apache.iceberg:iceberg-bom from 1.9.2 to 1.10.0 Sep 11, 2025
Copy link
Member

@devinrsmith devinrsmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Make sure nightlies pass before merging.

@malhotrashivam malhotrashivam merged commit e70f42c into deephaven:main Sep 11, 2025
24 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Sep 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants