-
Notifications
You must be signed in to change notification settings - Fork 17
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
feat: support temporal partition function #161
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, Thanks!
const DAY_PER_SECOND: f64 = 0.0000115741; | ||
const HOUR_PER_SECOND: f64 = 1_f64 / 3600.0; | ||
|
||
pub struct Year {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pub struct Year {} | |
pub struct Year; |
} | ||
} | ||
|
||
pub struct Month {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pub struct Month {} | |
pub struct Month; |
|
||
/// 1970-01-01 is base date in iceberg. | ||
/// 719163 is the number of days from 0000-01-01 to 1970-01-01 | ||
const BASE_DAY_FROM_CE: i32 = 719163; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const BASE_DAY_FROM_CE: i32 = 719163; | |
const EPOCH_DAYS_FROM_CE : i32 = 719163; |
} | ||
} | ||
|
||
pub struct Hour {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pub struct Hour {} | |
pub struct Hour; |
The ci can be fixed after #164 merged. |
No description provided.