Skip to content

Commit

Permalink
[Kernel] Add the InCommitTimestamp table feature (#3218)
Browse files Browse the repository at this point in the history
[Kernel] Add the `InCommitTimestamp` as supported writer table feature
  • Loading branch information
EstherBear authored Jun 5, 2024
1 parent a8e339f commit 18d1a27
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,8 @@ public static void validateReadSupportedTable(
* <ul>
* <li>protocol writer version 1.</li>
* <li>protocol writer version 2 only with appendOnly feature enabled.</li>
* <li>protocol writer version 7 with "appendOnly" feature enabled.</li>
* <li>protocol writer version 7 with {@code appendOnly}, {@code inCommitTimestamp-preview}
* feature enabled.</li>
* </ul>
*
* @param protocol Table protocol
Expand Down Expand Up @@ -110,6 +111,8 @@ public static void validateWriteSupportedTable(
// Only supported writer features as of today in Kernel
case "appendOnly":
break;
case "inCommitTimestamp-preview":
break;
default:
throw unsupportedWriterFeature(tablePath, writerFeature);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,10 @@ class TableFeaturesSuite extends AnyFunSuite {
checkSupported(createTestProtocol(minWriterVersion = 7))
}

test("validateWriteSupported: protocol 7 with appendOnly") {
checkSupported(createTestProtocol(minWriterVersion = 7, writerFeatures = "appendOnly"))
Seq("appendOnly", "inCommitTimestamp-preview").foreach { supportedWriterFeature =>
test(s"validateWriteSupported: protocol 7 with $supportedWriterFeature") {
checkSupported(createTestProtocol(minWriterVersion = 7, supportedWriterFeature))
}
}

Seq("invariants", "checkConstraints", "generatedColumns", "allowColumnDefaults", "changeDataFeed",
Expand Down

0 comments on commit 18d1a27

Please # to comment.