metrico / qryn

⭐️ All-in-One Polyglot Observability with OLAP Storage for Logs, Metrics, Traces & Profiles. Drop-in Grafana Cloud replacement compatible with Loki, Prometheus, Tempo, Pyroscope, Opentelemetry, Datadog and beyond :rocket:
https://qryn.dev
GNU Affero General Public License v3.0
1.24k stars 68 forks source link

Feat: bun support #352

Closed akvlad closed 1 year ago

akvlad commented 1 year ago

bun support

Breaking changes

  1. qryn.js is renamed to qryn.mjs according to the node standards.
  2. Node versions with no ES6 support (I believe v14) are deprecated

Known issues

Sometimes during the e2e test I have experienced the bun crash with the message of pure virtual function call:

pure virtual method called

uh-oh: A C++ exception occurred
bun will crash now 😭😭😭

----- bun meta -----
Bun v1.0.1 (31aec4eb) Linux x64 #34-Ubuntu SMP PREEMPT_DYNAMIC Mon Sep  4 13:06:55 UTC 2023
AutoCommand: 
Elapsed: 449470ms | User: 5805ms | Sys: 1363ms
RSS: 0.20GB | Peak: 4.78GB | Commit: 0.20GB | Faults: 0
----- bun meta -----

0   0x55f268efa3bb
1   ???
2   ???
3   ???
4   ???
5   ???
6   ???
7   ???
8   ???
9   ???
10  ???
11  ???
12  ???
13  ???
14  ???
15  ???
16  ???
17  ???
18  ???
19  ???
20  ???
21  ???
22  ???
23  ???
24  ???
25  ???

Crash report saved to:
  ~/.bun-crash/v1.0.1-1696593282967.crash

The crash is observed under different circumstances. This should be avoided via running qryn with pm2 or docker with auto-restart. The community help would be nice.

UPD: the issue seems fixed. The Bun worker module is not production ready. Will check for several next hours.

socket-security[bot] commented 1 year ago

New dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
@stricjs/router 4.2.9 None +0 35.2 kB reve-node