FB-1038 Move stat logging to ES client, log index names and types

cargomedia / cm

UNMAINTAINED - CM web application framework

MIT License

12 stars 18 forks source link

FB-1038 Move stat logging to ES client, log index names and types #2678

Closed tomaszdurka closed 7 years ago

tomaszdurka commented 7 years ago

Reverts cargomedia/cm#2677

fvovan commented 7 years ago

rest looks good, the only thing came to my mind is probably we don't even need Cluster class now. It's not a container anymore and all it does is just proxifying queries to Client (which is kind of proxy too). well also it allows to enable/disable cluster..

tomaszdurka commented 7 years ago

Yes, but I didn't want to refactor it here. Or maybe.

tomaszdurka commented 7 years ago

I will keep it, it has this "enabled" feature which we should keep for now.