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.

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

[Subtask] Add reserved properties to Table Properties when load an Iceberg table #2962

Closed
caican00 opened this issue Apr 16, 2024 · 0 comments · Fixed by #3511
Closed

[Subtask] Add reserved properties to Table Properties when load an Iceberg table #2962

caican00 opened this issue Apr 16, 2024 · 0 comments · Fixed by #3511
Assignees
Labels
0.5.1 Release v0.5.1 0.6.0 Release v0.6.0 subtask Subtasks of umbrella issue

Comments

@caican00
Copy link
Collaborator

Describe the subtask

Add reserved properties to Table Properties when load an Iceberg table, such as:

provider,
format,
current-snapshot-id,
location,
format-version,
sort-order,
identifier-fields

for example, when execute desc extended IcebergTableName,the result properties of this Iceberg table should contain the above properties.

Parent issue

#1571

@caican00 caican00 added the subtask Subtasks of umbrella issue label Apr 16, 2024
@jerryshao jerryshao added this to the Gravitino June Release milestone Apr 24, 2024
caican00 added a commit to caican00/gravitino that referenced this issue May 22, 2024
caican00 added a commit to caican00/gravitino that referenced this issue May 23, 2024
caican00 added a commit to caican00/gravitino that referenced this issue May 23, 2024
@FANNG1 FANNG1 added 0.5.1 Release v0.5.1 0.6.0 Release v0.6.0 labels May 23, 2024
FANNG1 pushed a commit that referenced this issue May 24, 2024
…ceberg Table Properties in Gravitino (#3511)

### What changes were proposed in this pull request?
Add reserved properties to Table properties when load an Iceberg table,
such as:

```
provider,
format,
current-snapshot-id,
location,
format-version,
sort-order,
identifier-fields
```

### Why are the changes needed?
for example, when execute `desc extended IcebergTableName`,it will get
some information from the result properties of this Iceberg table, so it
should contain the above properties.

Fix: #2962

### Does this PR introduce any user-facing change?
No.

### How was this patch tested?
New UTs and ITs.
github-actions bot pushed a commit that referenced this issue May 24, 2024
…ceberg Table Properties in Gravitino (#3511)

### What changes were proposed in this pull request?
Add reserved properties to Table properties when load an Iceberg table,
such as:

```
provider,
format,
current-snapshot-id,
location,
format-version,
sort-order,
identifier-fields
```

### Why are the changes needed?
for example, when execute `desc extended IcebergTableName`,it will get
some information from the result properties of this Iceberg table, so it
should contain the above properties.

Fix: #2962

### Does this PR introduce any user-facing change?
No.

### How was this patch tested?
New UTs and ITs.
mchades pushed a commit that referenced this issue May 24, 2024
…ceberg Table Properties in Gravitino (#3543)

### What changes were proposed in this pull request?
Add reserved properties to Table properties when load an Iceberg table,
such as:

```
provider,
format,
current-snapshot-id,
location,
format-version,
sort-order,
identifier-fields
```

### Why are the changes needed?
for example, when execute `desc extended IcebergTableName`,it will get
some information from the result properties of this Iceberg table, so it
should contain the above properties.

Fix: #2962

### Does this PR introduce any user-facing change?
No.

### How was this patch tested?
New UTs and ITs.

Co-authored-by: cai can <94670132+caican00@users.noreply.github.com>
diqiu50 pushed a commit to diqiu50/gravitino that referenced this issue Jun 13, 2024
…s to Iceberg Table Properties in Gravitino (apache#3511)

### What changes were proposed in this pull request?
Add reserved properties to Table properties when load an Iceberg table,
such as:

```
provider,
format,
current-snapshot-id,
location,
format-version,
sort-order,
identifier-fields
```

### Why are the changes needed?
for example, when execute `desc extended IcebergTableName`,it will get
some information from the result properties of this Iceberg table, so it
should contain the above properties.

Fix: apache#2962

### Does this PR introduce any user-facing change?
No.

### How was this patch tested?
New UTs and ITs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment