Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
e2b7ce1
Upgraded iceberg-java to 1.9.2, upgrade overridden classes, added sup…
subkanthi Apr 23, 2026
8856728
rollback changes to overridden classes.
subkanthi Apr 24, 2026
f898acc
Merge branch 'master' into iceberg_update_alter_table_null_default
subkanthi Apr 28, 2026
2eca2e7
rolled back changes to date/timestamp that was part of the original i…
subkanthi Apr 28, 2026
41f4e4f
Merge branch 'iceberg_update_alter_table_null_default' of https://git…
subkanthi Apr 28, 2026
acc5dfa
feat: Enhance alter-table command to support adding NOT NULL columns …
subkanthi Apr 29, 2026
468431a
Added integration test for data types
subkanthi May 2, 2026
1c03376
Restored deleted file
subkanthi May 2, 2026
2c31f00
Merge branch 'master' into iceberg_update_alter_table_null_default
subkanthi May 2, 2026
010613d
Integration test for basic data types.
subkanthi May 4, 2026
b0015ae
Integration test for basic data types.
subkanthi May 4, 2026
45b1c48
Merge branch 'iceberg_update_alter_table_null_default' of https://git…
subkanthi May 4, 2026
00b99d2
Add Datatypes test to verify
subkanthi May 28, 2026
b588867
Merge branch 'master' of https://github.com/Altinity/ice into iceberg…
subkanthi May 28, 2026
474ba10
Fix jackson exclusion
subkanthi May 28, 2026
4d42c3a
Fix jackson exclusion
subkanthi May 28, 2026
541c4f8
Removed function thats not used
subkanthi May 28, 2026
26e49c7
Moved user input JSON data type conversion logic to util class
subkanthi Jun 2, 2026
fda09e6
Merge branch 'master' of https://github.com/Altinity/ice into iceberg…
subkanthi Jun 2, 2026
0da65e8
Added Input Parser test
subkanthi Jun 2, 2026
fdfde19
Fix formatting errors
subkanthi Jun 2, 2026
52f5ace
Fixed data types test
subkanthi Jun 2, 2026
3cb74d7
Fixed datetime validation check
subkanthi Jun 2, 2026
7b5166a
Added UserInputParserTest
subkanthi Jun 2, 2026
b708a68
Change logic to throw schema version error if table is created using v2
subkanthi Jun 2, 2026
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/verify.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,6 @@ jobs:
run: >
./mvnw -pl ice-rest-catalog -am install -DskipTests=true -Pno-check &&
./mvnw -pl ice-rest-catalog failsafe:integration-test failsafe:verify
-Dit.test=DockerScenarioBasedIT,DockerLocalFileIOClickHouseIT
-Dit.test=DockerScenarioBasedIT,DockerLocalFileIOClickHouseIT,DockerLocalFileIOClickHouseAllTypesIT
-Ddocker.image=altinity/ice-rest-catalog:debug-with-ice-latest-master-amd64
-Dclickhouse.image=altinity/clickhouse-server:25.8.16.20002.altinityantalya
13 changes: 13 additions & 0 deletions ice-rest-catalog/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -244,6 +244,18 @@
<groupId>com.fasterxml.jackson.core</groupId>
<artifactId>jackson-core</artifactId>
</exclusion>
<exclusion>
<groupId>com.fasterxml.jackson.core</groupId>
<artifactId>jackson-databind</artifactId>
</exclusion>
<exclusion>
<groupId>com.fasterxml.jackson.core</groupId>
<artifactId>jackson-annotations</artifactId>
</exclusion>
<exclusion>
<groupId>com.fasterxml.jackson.datatype</groupId>
<artifactId>jackson-datatype-jsr310</artifactId>
</exclusion>
<exclusion>
<groupId>com.google.errorprone</groupId>
<artifactId>error_prone_annotations</artifactId>
Expand Down Expand Up @@ -577,6 +589,7 @@
<excludes>
<exclude>**/DockerScenarioBasedIT.java</exclude>
<exclude>**/DockerLocalFileIOClickHouseIT.java</exclude>
<exclude>**/DockerLocalFileIOClickHouseAllTypesIT.java</exclude>
</excludes>
</configuration>
</plugin>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
import java.util.concurrent.atomic.AtomicReference;
import java.util.function.Function;
import java.util.function.Predicate;
import org.apache.iceberg.BaseMetastoreOperations.CommitStatus;
import org.apache.iceberg.encryption.EncryptionManager;
import org.apache.iceberg.exceptions.AlreadyExistsException;
import org.apache.iceberg.exceptions.CommitFailedException;
Expand Down Expand Up @@ -294,20 +295,39 @@ public long newSnapshotId() {
* were attempting to set. This is used as a last resort when we are dealing with exceptions that
* may indicate the commit has failed but are not proof that this is the case. Past locations must
* also be searched on the chance that a second committer was able to successfully commit on top
* of our commit.
* of our commit. When the {@code newMetadataLocation} is not found, the method returns {@link
* CommitStatus#UNKNOWN}.
*
* @param newMetadataLocation the path of the new commit file
* @param config metadata to use for configuration
* @return Commit Status of Success, Failure or Unknown
* @return Commit Status of Success, Unknown
*/
protected CommitStatus checkCommitStatus(String newMetadataLocation, TableMetadata config) {
return CommitStatus.valueOf(
checkCommitStatus(
tableName(),
newMetadataLocation,
config.properties(),
() -> checkCurrentMetadataLocation(newMetadataLocation))
.name());
return checkCommitStatus(
tableName(),
newMetadataLocation,
config.properties(),
() -> checkCurrentMetadataLocation(newMetadataLocation));
}

/**
* Attempt to load the table and see if any current or past metadata location matches the one we
* were attempting to set. This is used as a last resort when we are dealing with exceptions that
* may indicate the commit has failed but are not proof that this is the case. Past locations must
* also be searched on the chance that a second committer was able to successfully commit on top
* of our commit. When the {@code newMetadataLocation} is not found, the method returns {@link
* CommitStatus#FAILURE}.
*
* @param newMetadataLocation the path of the new commit file
* @param config metadata to use for configuration
* @return Commit Status of Success, Failure or Unknown
*/
protected CommitStatus checkCommitStatusStrict(String newMetadataLocation, TableMetadata config) {
return checkCommitStatusStrict(
tableName(),
newMetadataLocation,
config.properties(),
() -> checkCurrentMetadataLocation(newMetadataLocation));
}

/**
Expand Down
Loading
Loading