Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Delta Protocol] Follow up comments for ClusteringTable feature #2294

Closed
wants to merge 8 commits into from
+47 −8
Closed

[Delta Protocol] Follow up comments for ClusteringTable feature #2294

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
c31158e
fix comments
dabao521 Nov 9, 2023
058e4f1
address comments
dabao521 Nov 14, 2023
d7f400d
follow up comments
dabao521 Nov 15, 2023
6689304
Let clustering table feature added either during creation or later stage
dabao521 Nov 15, 2023
adec52b
fix comments
dabao521 Nov 16, 2023
0f7a442
fix comment
dabao521 Nov 16, 2023
48402f4
add back partitionValues
dabao521 Nov 16, 2023
1695e92
remove extra size
dabao521 Nov 16, 2023
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
Prev Previous commit
remove extra size
@dabao521
dabao521 committed Nov 16, 2023
commit 1695e92522148a5b28067947fac6f11bc9376131
1 change: 0 additions & 1 deletion PROTOCOL.md
View file
Original file line number Diff line number Diff line change
@@ -433,7 +433,6 @@ The following is an example `add` action for a clustered table:
"path": "date=2017-12-10/part-000...c000.gz.parquet",
"partitionValues": {},
"size": 841454,
"size": 841454,
"modificationTime": 1512909768000,
"dataChange": true,
"baseRowId": 4071,