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

readme suggests to use graphite_index but default config is generated with graphite_tree #82

Open
kt97679 opened this issue Jan 28, 2021 · 3 comments
Assignees

Comments

@kt97679
Copy link

kt97679 commented Jan 28, 2021

Hello.

README.md has guidelines to setup graphite_index table and use it in the config of the carbon-clickhouse. From what I see carbon/config/go uses graphite_tree instead. Default config generated by carbon-clickhouse is not using graphite_index as well. README.md for the graphite-clickhouse has a note that graphite_tree is deprecated.

May I ask you to clarify what table should be used: graphite_index or graphite_tree?

Thanks!

@Felixoid
Copy link
Collaborator

Felixoid commented Jan 29, 2021

Hey, the readme is correct, index table should be used https://github.com/lomik/graphite-clickhouse/wiki/IndexTable

It makes sense for sure to change the defaults

@kt97679
Copy link
Author

kt97679 commented Jan 29, 2021

Thanks a bunch for the clarification @Felixoid !

@Felixoid Felixoid self-assigned this May 10, 2021
@Felixoid
Copy link
Collaborator

After merging go-graphite/graphite-clickhouse#157 I'll rework the config for carbon-clickhouse as well

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

No branches or pull requests

2 participants