vicanso / influxdb-nodejs

Simple influxdb client
https://vicanso.github.io/influxdb-nodejs/
MIT License
88 stars 22 forks source link

Comparison with more popular InfluxDB libraries #7

Closed dandv closed 7 years ago

dandv commented 7 years ago

Thanks for the new InfluxDB library. It would help to document the motivation for writing it, and how it compares with the more established Node.js libraries for InfluxDB:

vicanso commented 7 years ago

@dandv I will compare this project with the other two modules to find out the differences later.

dandv commented 7 years ago

@vicanso: Thank you. Did you take a look at at the other projects before starting this one, or maybe you just went ahead and developed it for your own projects' needs?

vicanso commented 7 years ago

@dandv I am sorry for forget to reply you. Yes I did look at the other projects before. I think those projects are not convenient.

dandv commented 7 years ago

@vicanso: can you elaborate how the other libraries, and especially the official client, are not convenient?

sveip commented 7 years ago

Too me it looks like the official library and it's doc has improved allot. When this library was created that was not the case.

https://node-influx.github.io/manual/tutorial.html

Although the usage manual is still quite a bit lacking https://node-influx.github.io/manual/usage.html

Just my 2 cents.

On 2 Dec 2016 6:40 a.m., "Dan Dascalescu" notifications@github.com wrote:

@vicanso https://github.com/vicanso: can you elaborate how the other libraries, and especially the official client https://docs.influxdata.com/influxdb/v1.1/tools/api_client_libraries/#javascript-node-js, are not convenient?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/vicanso/influxdb-nodejs/issues/7#issuecomment-264377967, or mute the thread https://github.com/notifications/unsubscribe-auth/AFceL4X6LLlW4Xl46HiaV2jBJkDq8ttHks5rD68tgaJpZM4KGCbz .

vicanso commented 7 years ago

@dandv Since node-influx uses influx ql to query data, I don't want to write select * from http where time >= -2h and time <= -1h. I prefer querying data in this way:

client.query('http')
  .set({
    'start': '-2h',
    'end': '-1h',
  })

@sveip I write the project in my spare time . Honestly, I suggest you to choose the official library for commercial purpose. I start this project just because I want to write something in my spare time. It's not perfect, even contains flaws, but it's mine.

sveip commented 7 years ago

Thanks for the advice,

I think I'll stick to yours, I really like it ☺️

On 3 Dec 2016 11:03 a.m., "Tree Xie" notifications@github.com wrote:

@dandv https://github.com/dandv Since node-influx uses influx ql to query data, I don't want to write select * from http where time >= -2h and time <= -1h. I prefer querying data in this way:

client.query('http') .set({ 'start': '-2h', 'end': '-1h', })

@sveip https://github.com/sveip I write the project in my spare time . Honestly, I suggest you to choose the official library for commercial purpose. I start this project just because I want to write something in my spare time. It's not perfect, even contains flaws, but it's mine.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/vicanso/influxdb-nodejs/issues/7#issuecomment-264629858, or mute the thread https://github.com/notifications/unsubscribe-auth/AFceL2jpavkCxkmRaQypWPbwjfWTlA2Nks5rET5pgaJpZM4KGCbz .

vicanso commented 7 years ago

@dandv @sveip

API was written.