Skip to content

refactor: remove safe_cast_record_values from BigQuery streaming destinations#55

Merged
anaselmhamdi merged 1 commit intobizon-data:mainfrom
gorgias:gorgias
Jan 6, 2026
Merged

refactor: remove safe_cast_record_values from BigQuery streaming destinations#55
anaselmhamdi merged 1 commit intobizon-data:mainfrom
gorgias:gorgias

Conversation

@anaselmhamdi
Copy link
Collaborator

Remove automatic type casting for unnested records in BigQuery streaming destinations. Users should now use transforms to handle type conversions (dict/list to JSON, integer timestamps to datetime strings) when using unnest=True.

Changes:

  • Remove safe_cast_record_values method from both destinations
  • Remove unused datetime import
  • Update tests to use current config schema (record_schemas instead of deprecated table_id/record_schema)
  • Remove tests for protobuf errors that don't apply to legacy streaming API

🤖 Generated with Claude Code

Summary

Type of Change

  • Bug fix
  • New feature
  • New connector (source/destination)
  • Documentation
  • Refactoring
  • Other: ___

Changes Made

Testing

  • Unit tests added/updated
  • Manual testing performed
  • Existing tests pass (uv run pytest)

Checklist

  • Code follows project style (make format)
  • Self-review completed
  • Documentation updated (if needed)

Related Issues

…inations (#104)

Remove automatic type casting for unnested records in BigQuery streaming
destinations. Users should now use transforms to handle type conversions
(dict/list to JSON, integer timestamps to datetime strings) when using
unnest=True.

Changes:
- Remove safe_cast_record_values method from both destinations
- Remove unused datetime import
- Update tests to use current config schema (record_schemas instead of
  deprecated table_id/record_schema)
- Remove tests for protobuf errors that don't apply to legacy streaming API

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
@anaselmhamdi anaselmhamdi self-assigned this Jan 6, 2026
@anaselmhamdi anaselmhamdi merged commit 8890121 into bizon-data:main Jan 6, 2026
1 check 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.

1 participant