Skip to content

[Feature] support hour time unit with dynamic parition - #4514

Merged
morningman merged 1 commit into
apache:masterfrom
caoyang10:master
Sep 6, 2020
Merged

[Feature] support hour time unit with dynamic parition#4514
morningman merged 1 commit into
apache:masterfrom
caoyang10:master

Conversation

@caoyang10

Copy link
Copy Markdown
Contributor

Many tables are so large that need seperate partitions with "HOUR" time unit. But now dynamic partition doesn't support "HOUR" time unit and it was marked as "TODO". So I support the feature and it works.

@morningmanmorningman added area/catalog Issues or PRs related to catalog management kind/improvement labels Sep 5, 2020

@morningmanmorningman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@morningmanmorningman added the approved Indicates a PR has been approved by one committer. label Sep 5, 2020
@morningman
morningman merged commit 03d9f6d into apache:masterSep 6, 2020
caoyang10 added a commit to caoyang10/incubator-doris that referenced this pull request Oct 29, 2020
Many tables are so large that need seperate partitions with "HOUR" time unit.
But now dynamic partition doesn't support "HOUR" time unit and it was marked as "TODO".
So I support the feature and it works.
@yangzhgyangzhg mentioned this pull request Feb 9, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approvedIndicates a PR has been approved by one committer.area/catalogIssues or PRs related to catalog managementkind/improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@caoyang10@morningman