Skip to content

Conversation

@qidaye
Copy link
Contributor

@qidaye qidaye commented Feb 8, 2022

Proposed changes

Issue Number: close #7980

Problem Summary:

  1. Add iceberg_table_creation_strict_mode in fe.conf to control iceberg external table creation, when data type is not supported in Doris.
  2. Add REFRESH syntax to synchronize the Iceberg table and database.
  3. Support create Iceberg external table with specific column definitions.

Checklist(Required)

  1. Does it affect the original behavior: (Yes/No/I Don't know) No
  2. Has unit tests been added: (Yes/No/No Need)
  3. Has document been added or modified: (Yes/No/No Need) Yes
  4. Does it need to update dependencies: (Yes/No) No
  5. Are there any changes that cannot be rolled back: (Yes/No) No

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

qijianliang01 added 2 commits February 8, 2022 14:55
Change-Id: I75f8d54db9bd3729d0b67c80b53cc22ee90bc65b
Map<String, String> dbProperties = db.getDbProperties().getProperties();
// build iceberg properties
db.getDbProperties().addAndBuildProperties(dbProperties);
// 1. check database type
Copy link
Contributor

Choose a reason for hiding this comment

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

Why calling addAndBuildProperties here?


// 2. drop database
DropDbStmt dropDbStmt = new DropDbStmt(true, dbName, true);
catalog.dropDb(dropDbStmt);
Copy link
Contributor

Choose a reason for hiding this comment

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

There may be other type of tables in this database. If you drop it, all other tables will lost.

Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

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

LGTM

@morningman morningman added area/iceberg kind/feature Categorizes issue or PR as related to a new feature. labels Feb 9, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2022

PR approved by at least one committer and no changes requested.

@github-actions github-actions bot added approved Indicates a PR has been approved by one committer. reviewed labels Feb 9, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2022

PR approved by anyone and no changes requested.

@morningman morningman merged commit 92b690f into apache:master Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. area/iceberg kind/feature Categorizes issue or PR as related to a new feature. reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Support synchronize remote Iceberg table and dababase

2 participants