Dan Hansen

@rahim Late reply, but we were modifying a duped version of the params so that the input would remain compatible for the form helpers @jonatack Thanks, I'll take a look...

it seems silly for me to speak on goccy's behalf, but in my experience contributing over the last few months, they only have time to review and do releases every...

It seems to me like the BigQuery client is retrying your query as the server closed the connection. Judging by the timestamps in your logs, I'm guessing you're running into...

This should be closed now that #266 has been merged.

Did the new release fix your issue? @Cililing

It seems that this will require a change to the Go ZetaSQL bindings. I'm not familiar with that process, @goccy could you take a look? https://github.com/goccy/go-zetasql/issues/27

I also see the zetasql warnings when debugging in GoLand, I don't have much more to say on this than that! 🤔

It seems to me that the `PATCH /projects/{project]/datasets/{dataset}/tables/{table}` endpoint has numerous problems. [We currently decode the request body into a `bigquery.Table`](https://github.com/goccy/bigquery-emulator/blob/main/server/handler.go#L2685-L2689). For example, if we are updating the description, we'd...

Yes, we should update `go-zetasqlite` to format the query correctly. The ZetaSQL AST includes `ResolvedCreateTableAsSelectStmt.output_column_list()` which should give us what we need to format it into SQLite. https://github.com/google/zetasql/blob/589026c410c42de9aa8ee92ad16f745977140041/docs/resolved_ast.md#resolvedcreatetableasselectstmt Here's the...

We'll also need to enable the language feature in `analyzer.go` https://github.com/goccy/go-zetasql/blob/d769c5827652fccc561d80ad3292d54782c7cbfd/enum.go#L173-L177