Skip to content

chore: clean up compiler warnings#12848

Open
EdanStasiuk wants to merge 5 commits intoneondatabase:mainfrom
EdanStasiuk:clean-up-compiler-warnings
Open

chore: clean up compiler warnings#12848
EdanStasiuk wants to merge 5 commits intoneondatabase:mainfrom
EdanStasiuk:clean-up-compiler-warnings

Conversation

@EdanStasiuk
Copy link
Copy Markdown

@EdanStasiuk EdanStasiuk commented Jan 11, 2026

Problem

ref: #12847 - Compiler warnings being generated in remote_storage

Summary of changes

Within libs/remote_storage/src/gcs_bucket.rs:

  • Remove unused imports
  • Prepend unused variables with _
  • Remove dead max_keys_per_delete function from impl GCSBucket (keep only trait implementation in RemoteStorage)
  • Fix issues flagged by rustfmt, and apply Clippy suggestions to improve code idioms

- Removed unused private `max_keys_per_delete` in `impl GCSBucket`
- Kept trait declaration in `lib.rs` and its implementation in `impl RemoteStorage`
@EdanStasiuk EdanStasiuk requested a review from a team as a code owner January 11, 2026 01:02
@EdanStasiuk EdanStasiuk force-pushed the clean-up-compiler-warnings branch 2 times, most recently from ed2d0c8 to 71ed827 Compare January 28, 2026 01:59
@EdanStasiuk
Copy link
Copy Markdown
Author

@erikgrinaker this PR is ready for review when you have a chance. Thanks!

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