Skip to content

refactor: rename S3 bucket, relocate tests, update deps#56

Closed
zer0stars wants to merge 1 commit intoicebergfrom
batching
Closed

refactor: rename S3 bucket, relocate tests, update deps#56
zer0stars wants to merge 1 commit intoicebergfrom
batching

Conversation

@zer0stars
Copy link
Member

Summary

  • Rename ICEBERG_WAREHOUSE S3 bucket from dimo-iceberg-* to dimo-storage-* in stream configs and Helm values (dev + prod)
  • Fix import path (cloudeventparquet/encodersencoders) and relocate tests: parseBucket tests moved to cloudeventtoparquet, RawEvent_DataBase64 moved to encoders; delete orphaned parquetwriter_test.go
  • Update Go deps: add aws-sdk-go-v2 core + S3, drop segmentio/encoding, pin tidwall/match v1.1.1

Test plan

  • go test ./... passes
  • Benthos lint (make lint-benthos) passes with updated stream configs
  • Verify dev/prod deployments resolve the new S3 bucket name correctly

Made with Cursor

- Rename ICEBERG_WAREHOUSE bucket from dimo-iceberg-* to dimo-storage-*
  across stream configs and Helm values (dev + prod)
- Fix import path: cloudeventparquet/encoders -> encoders
- Move parseBucket tests into cloudeventtoparquet package and
  RawEvent_DataBase64 test into encoders; delete parquetwriter_test.go
- Update go.mod: add aws-sdk-go-v2 core + s3, drop segmentio/encoding,
  pin tidwall/match v1.1.1
- Fix struct field alignment in icebergcommit_test.go

Co-authored-by: Cursor <cursoragent@cursor.com>
@zer0stars zer0stars closed this Feb 16, 2026
@zer0stars zer0stars deleted the batching branch February 16, 2026 22:45
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

Comments