Riley Thompson

Results 26 comments of Riley Thompson

Does this still need to be worked on?

@mungo-pod See [this comment](https://github.com/supabase/supabase-js/issues/936#issuecomment-2027940513) above, the override needs to be on `"@supabase/auth-js"` now.

Repro with 0.9.0: ``` $ clj -Sdeps '{:deps {com.clickhouse/clickhouse-jdbc {:mvn/version "0.9.0"}}}' user=> (let [conn (java.sql.DriverManager/getConnection "jdbc:clickhouse://localhost:8123/my-db" "default" "") stmt (.prepareStatement conn "SELECT * FROM `my-table`;")] (let [rs (.executeQuery stmt)] (while...

Unable to reproduce: ``` $ sqlite3 gencol.db sqlite> CREATE TABLE test_gencol ( a INTEGER PRIMARY KEY, b INT, c TEXT, d INT GENERATED ALWAYS AS (a*abs(b)) VIRTUAL, e TEXT GENERATED...

LGTM. A simple test for the prometheus metric might be good. To confirm my understanding, if it was weak map was growing too large and the speculative fix of using...

@paoliniluis I was able to prompt claude to some sort of a repro: `Caused by: java.lang.IllegalArgumentException: Required data attribute 'ROWS_COUNT' does not exist` But I haven't looked too closely at...

Moving this to a P2 for now as it requires a specific configuration and has potential workarounds

The slack link in the description is broken for me but this one isn't: https://metaboat.slack.com/archives/C05NXACAG1G/p1764951661272479

The issue is `formatSQL` in `engine.ts` adding new lines in front of clauses without checking if they are commented

Reproduced on stats. Issue seems to be that we are generating the full name column twice: ``` " \"public\".\"v_content\".\"event_timestamp\" AS \"event_timestamp\",", " \"v_users__via__creator_id\".\"full_name\" AS \"v_users__via__creator_id__full_name\",", " \"v_users__via__creator_id\".\"full_name\" AS \"v_users__via__creator_id__full_name\"", "...