Skip to content
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

Update README.md #854

Merged
merged 5 commits into from
Nov 19, 2024
Merged

Update README.md #854

merged 5 commits into from
Nov 19, 2024

Conversation

sarwarbhuiyan
Copy link
Collaborator

PR checklist:

  • Did you run ClangFormat ?
  • Did you separate headers to a different section in existing community code base ?
  • Did you surround proton: starts/ends for new code in existing community code base ?

Please write user-readable short description of the changes:

@CLAassistant
Copy link

CLAassistant commented Nov 18, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@jovezhong jovezhong left a comment

Choose a reason for hiding this comment

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

thanks for the refining. Comments added

@jovezhong
Copy link
Contributor

Hi @sarwarbhuiyan , since you are updating the README ,maybe find a proper place to mention the difference of Timeplus and ClickHouse https://www.timeplus.com/timeplus-and-clickhouse. The link is to comparing the MV in Timeplus and ClickHouse, also appliable to the open source engine

@jovezhong jovezhong merged commit 905a743 into develop Nov 19, 2024
1 check passed
@jovezhong jovezhong deleted the sarwarbhuiyan-patch-1 branch November 19, 2024 18:02
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