Skip to content

Adding DateTime64 as an available type#520

Closed
trolley813 wants to merge 2 commits intoClickHouse:masterfrom
Clover-Group:master
Closed

Adding DateTime64 as an available type#520
trolley813 wants to merge 2 commits intoClickHouse:masterfrom
Clover-Group:master

Conversation

@trolley813
Copy link

Pull request for fixing #517 .

@zhicwu
Copy link
Contributor

zhicwu commented Mar 4, 2021

Thanks @trolley813 for the pull request. I think we also need to consider: 1) DateTime64 contains nano seconds, and 2) DateTime can represent both DateTime32 and DateTime64. I'm working on a new pull request to support missing data types, which will address these issues. I hope I can submit tonight.

@zhicwu zhicwu closed this Mar 4, 2021
@zhicwu
Copy link
Contributor

zhicwu commented Mar 5, 2021

Will merge PR #589 to close this.

@zhicwu zhicwu linked an issue Mar 5, 2021 that may be closed by this pull request
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.

DateTime64 as ClickHouseDataType

2 participants