xephonhq / xephon-b

A time series database benchmark suite
https://xephonhq.github.io/xephon-b/
MIT License
8 stars 3 forks source link

[partial] TSDB Proxy Client #19

Closed at15 closed 7 years ago

at15 commented 7 years ago

Related #15 #18

Util

Improve docker image

Command

KairosDB

InfluxDB

OpenTSDB

at15 commented 7 years ago

the thing is, I lost some commit for testing the KairosDB client because I was thinking about adding a flag so tests require KairosDB running can be disabled on travis. So I didn't push them to remote and re installed my PC. The lost part should be a method to generate test payload and a test to query the test payload against a running KairosDB, also I noticed the at least one tag issue from KairosDB

at15 commented 7 years ago

Got error when try to use the flag checked added in 0046976, it causes an important cycle, and actually this test is no longer a unit test, it's an integration test that test most functions of KairosDBHTTPClient, (could be e2e maybe) so I think I need a sub package called integration inside kairosdb or just instead tsdb

at15 commented 7 years ago

LGTM

at15 commented 7 years ago

LGTM

Though most goal of this PR is not finished., but they should be split to more fine grained PR

Approved with PullApprove