search

scylladb / alternator-load-balancing

Various tricks, scripts, and libraries, for load balancing multiple Alternator nodes
Apache License 2.0
18 stars 11 forks source link

Remove excessive printouts #19

Closed nyh closed 6 months ago

nyh commented 6 months ago

This patch removes excessive printouts on the Go, Javascript and Python drivers. While printing every decision on every request is pretty for demonstrating the load balancing, it is annoying when used in production and can needlessly fill the logs.

As far as I can tell, the Java driver did not have such excessive printouts (it has logs in the FINE level, but those will be normally not visible).

tzach commented 6 months ago

@nyh look like we have 3 approvales. Whats the next step?

nyh commented 6 months ago

@nyh look like we have 3 approvales. Whats the next step?

I don't know, I guess I'll just have to merge my own patch.

mykaul commented 6 months ago

I can click the merge if you wish. Who are the maintainers of this repo?