-
Notifications
You must be signed in to change notification settings - Fork 89
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
combine chproxy and clickhouse-bulk #20
Comments
How I can help you? You want import clickhouse-bulk as module or implement this functionality? |
thanks for your reply, I want to implement this functionality |
As example, you can add bulk before chproxy |
I have a different package and I'd love to import clickhouse-bulk to do exactly the same (while kipping the interface slim). Would you be interested in that sort of refactoring, and consider a PR perhaps? |
What do you want to change? Can you send pr or needed changes? |
Возможно стоит по русски) У меня сейчас не особо много времени делать какой-то большой объем работе. Но если изменения небольшие, могу сделать. Если что-то большое, то надо смотреть, что нужно. |
Basically I wanted a plug and play solution to use to batch up calls the client, e.g via https://github.com/mailru/dbr . I ended basically mimicking the Do you think something like that has place in clickbouse-bulk? |
I think that such solutions are quite private. And yet I do not understand what should result? It seems to me that collecting requests from structures is much faster than processing it with a parser in collector.go. |
I want to add bulk features to chproxy,can you help me how to do it?
The text was updated successfully, but these errors were encountered: