metrico / qryn

Polyglot Observability Stack. Lightweight & Drop-in compatible with Loki, Prometheus, Tempo, Pyroscope, Opentelemetry, Datadog & more! WASM powered ⭐️ Star to Support
https://qryn.dev
GNU Affero General Public License v3.0
1.05k stars 63 forks source link

TypeError: undefined is not an object (evaluating 'clickhouseOptions.queryOptions') #482

Closed bzed closed 3 months ago

bzed commented 3 months ago

Hi,

seems there was a bug in #361 -

54 |       return
55 |     }
56 |     const _queue = this.queue
57 |     this.queue = []
58 |     await rawRequest(this.statement, _queue.join('\n'), DATABASE_NAME(), { maxBodyLength: Infinity })
59 |   }
                                                                                     ^
TypeError: undefined is not an object (evaluating 'clickhouseOptions.queryOptions')
      at /app/lib/db/throttler.js:59:81
      at anonymous (native:1:1)
      at /app/lib/db/clickhouse.js:92:9
      at anonymous (native:1:1)
      at /app/lib/handlers/push.js:24:7

I think https://github.com/metrico/qryn/blob/b4cda9e656f6720fe5ee7eaccb9dcd2c12e6ac82/lib/db/throttler.js#L3 should require clickhouse_options now?

Thanks,

Bernd

akvlad commented 3 months ago

@bzed apparently a bun problem. Will be fixed today. Please use the NodeJS version for now.

akvlad commented 3 months ago

@bzed everything fixed on 3.2.16 . Sorry for the inconvenience.

lmangani commented 3 months ago

Closing are resolved, please feel free to reopen if needed and thanks for the report!