Skip to content

Bump spirit to v0.12.1 and Go toolchain to 1.26.1 - #6

Merged
morgo merged 1 commit into
block:mainfrom
morgo:bump-spirit-latest
Apr 15, 2026
Merged

Bump spirit to v0.12.1 and Go toolchain to 1.26.1#6
morgo merged 1 commit into
block:mainfrom
morgo:bump-spirit-latest

Conversation

@morgo

@morgo morgo commented Apr 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

Bump dependencies:

  • github.com/block/spirit v0.10.1 → v0.12.1-0.20260414195601-d7c75c14cdfa
  • Go toolchain 1.25.3/1.25.6 → 1.26.1

Changes

Spirit renamed TableInfo.QuotedName (field containing backtick-quoted `schema`.`table`) to QuotedTableName (backtick-quoted `table` only, without schema prefix). Updated all references in:

  • pkg/stage/stager.go (7 occurrences)
  • pkg/archive/buffer_stager.go (2 occurrences)

This is needed as a transitive dependency for other changes internally.

@morgo
morgo force-pushed the bump-spirit-latest branch from 3c283a5 to 40367ff Compare April 15, 2026 03:19
Bump dependencies:
- github.com/block/spirit v0.10.1 → v0.12.1-0.20260414195601-d7c75c14cdfa
- Go toolchain 1.25.3/1.25.6 → 1.26.1
- golangci-lint 2.8.0 → 2.11.4 (built with Go 1.26.1)
- Dockerfile golang:1.25 → golang:1.26

Spirit API changes:
- TableInfo.QuotedName (backtick-quoted `schema`.`table`) was replaced
  by QuotedTableName (backtick-quoted `table` only, without schema).
  Update all references in pkg/stage/stager.go and pkg/archive/buffer_stager.go.
- Because QuotedTableName no longer includes the schema prefix, internal
  spirit operations like ANALYZE TABLE now run against the connection's
  default database. Remove the unnecessary SetInfo() calls from
  pkg/audit/db.go where the audit schema differs from the connection's
  default database. The SetInfo() results were never used — the TableInfo
  was discarded immediately after creation.
@morgo
morgo force-pushed the bump-spirit-latest branch from 40367ff to be7280b Compare April 15, 2026 03:27
@morgo
morgo marked this pull request as ready for review April 15, 2026 04:07
@morgo
morgo merged commit 53087f4 into block:main Apr 15, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants