Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants