Async response

[gianarb]

Hello!

I am working on integration between Zipkin and InfluxDB with @goller these days.
At the moment we are using this library. We would like to have back the async response other than the sync one as it happens now.

You can see why here in this comment

openzipkin/zipkin#1781 (comment)

What do you think about exposing this capability maybe creating a sync and async adapter in the client itself?

Thanks a lot

[majst01]

Nice to see the integration into zipkin !
I'm happy to review PRs.

[gianarb]

Please do @majst01 I am not a java dev and I am doing my best to look like one! :D

[majst01]

I have not spare time actually to invest a lot into this library, sorry.

[timhallinflux]

@ivankudibal -- is this something your team can work on?

[dubsky]

@gianarb -- Do you have batching enabled ? When you do so, influxdb client will collect the data points into a buffer and it will flush it automatically in a separate thread when certain properties of the entries in the buffer will be met (count, age).

For queries there are methods that are asynchronous:

InfluxDb.java:
public void query(final Query query, final Consumer onSuccess, final Consumer onFailure);

[fmachado]

@gianarb I added some comments directly to the mentioned zipkin PR.

[gianarb]

I didn't know about this method. I will have a look. Thanks

public void query(final Query query, final Consumer onSuccess, final Consumer onFailure);

We spoke about batch writing but it's something that zipkin can manage by itself and I am not sure if having 2 resource managers will be good. At the moment we decided to avoid baching.

Thanks @fmachado !!