Skip to content

Conversation

@eschultink
Copy link
Member

@eschultink eschultink commented Jan 1, 2026

Fixes

  • add some @Serial to avoid ide warnings
  • fix various warnings seen in maven builds, to streamline those

Change implications

hopefully nothing; ensure tests pass


Note

Focuses on warning cleanup and minor API modernizations across modules; no functional changes intended.

  • Add @Serial serialVersionUID to multiple Serializable types (StorageHandler.ObjectTransform, ColumnarRules, JsonSchemaFilterUtils.Options, WebhookCollectionRules and nested classes)
  • Prefer Objects.equals over StringUtils.equals for gzip checks and use Objects.requireNonNullElse for header merging in AWS request adapters
  • Migrate CSV parsing to CSVFormat.DEFAULT.builder().get() and align header handling in ColumnarBulkDataSanitizerImpl and RecordBulkDataSanitizerImpl
  • Replace RandomUtils with ThreadLocalRandom for sampling in GCP handler; keep ~1% warning sampling
  • Provide AWS lambdaEventMapper via JsonMapper.builder() with case-insensitive properties
  • Import/order cleanup and minor refactors (no behavior changes expected)

Written by Cursor Bugbot for commit ef41b2d. This will update automatically on new commits. Configure here.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses build warnings by adding @Serial annotations to serializable classes and replacing deprecated Apache Commons and Jackson APIs with their modern equivalents. The changes modernize code to eliminate IDE and Maven build warnings without altering functionality.

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
GcpApiDataRequestHandler.java Replaces deprecated RandomUtils with ThreadLocalRandom
APIGatewayV2HTTPEventRequestAdapter.java Replaces deprecated ObjectUtils.defaultIfNull with Objects.requireNonNullElse and reorganizes imports
APIGatewayV1ProxyEventRequestAdapter.java Replaces deprecated ObjectUtils.defaultIfNull with Objects.requireNonNullElse and reorganizes imports
AwsModule.java Refactors ObjectMapper creation to use JsonMapper builder pattern
FieldTransform.java Removes incorrect @nonnull annotation from primitive boolean field and reorganizes imports
WebhookCollectionRules.java Adds @serial annotation and reorganizes imports
RecordRules.java Reorganizes imports
JsonSchemaFilterUtils.java Adds @serial annotation to serialVersionUID
Endpoint.java Reorganizes imports
ColumnarRules.java Adds @serial annotation and reorganizes imports
RecordBulkDataSanitizerImpl.java Replaces deprecated CSVFormat methods with builder pattern and reorganizes imports
ColumnarBulkDataSanitizerImpl.java Replaces deprecated CSVFormat methods with builder pattern and reorganizes imports
StorageHandler.java Replaces StringUtils.equals with Objects.equals for null-safe comparison and adds @serial annotation
ClientCredentialsGrantTokenRequestBuilder.java Reorganizes imports

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cursor
Copy link

cursor bot commented Jan 4, 2026

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on February 10.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@eschultink eschultink enabled auto-merge (squash) January 4, 2026 16:24
@eschultink eschultink merged commit 55963e7 into rc-v0.5.16 Jan 4, 2026
47 checks passed
@eschultink eschultink deleted the s215-build-warnings branch January 4, 2026 16:26
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.

3 participants