Skip to content

[HUDI-2676] Hudi should synchronize owner information to hudi _rt/_ro…#3911

Merged
leesf merged 1 commit intoapache:masterfrom
xiarixiaoyao:ow
Nov 3, 2021
Merged

[HUDI-2676] Hudi should synchronize owner information to hudi _rt/_ro…#3911
leesf merged 1 commit intoapache:masterfrom
xiarixiaoyao:ow

Conversation

@xiarixiaoyao
Copy link
Contributor

@xiarixiaoyao xiarixiaoyao commented Nov 3, 2021

… table.

Tips

What is the purpose of the pull request

hudi rt/ro table missing owner information, 
 If permission protection is enabled for the query engine, the verification of the protection group will fail。

Verify this pull request

beeline
desc formatted tcc_rt;

after patch:

| Database: | default | NULL |
| OwnerType: | USER | NULL |
| Owner: | xxxx | NULL |
| CreateTime: | Tue Nov 02 19:27:59 CST 2021 | NULL

Committer checklist

  • Has a corresponding JIRA in PR title & commit

  • Commit message is descriptive of the change

  • CI is green

  • Necessary doc changes done or have another open PR

  • For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@xiarixiaoyao
Copy link
Contributor Author

@leesf @nsivabalan could you pls help me review this code thanks.
a minor fix, we need synchronize owner information to hudi _rt/_ro table.

@hudi-bot
Copy link
Collaborator

hudi-bot commented Nov 3, 2021

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run travis re-run the last Travis build
  • @hudi-bot run azure re-run the last Azure build

@leesf leesf merged commit acc4062 into apache:master Nov 3, 2021
@xiarixiaoyao xiarixiaoyao deleted the ow branch December 3, 2021 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants