Adding SqsBatchManager interfaces and Refactoring core BatchManager with interfaces

[16lim21]

Motivation and Context

I added interfaces for the core BatchManager to abstract away the private methods in the implementation. I also created the interfaces and class files for the SqsBatchManager to prepare for implementation.

Description

I added some interfaces in a new batchmanager package that is not internal and abstracts away the implementation details for the underlying DefaultBatchManager. Customers will therefore interact with the interfaces and classes in the public batchmanager package as opposed to the internal one.

Meanwhile, I modified the BatchOverrideConfiguration class to accept optional values, which would be filled in with default values by a core BatchConfiguration class. Services that implement a BatchManager will not need to rely on this BatchConfiguration class, it is just there to ensure default values are provided in case customers implement their own service-specific batch manager.

Finally, I added some classes and interfaces for the SqsBatchManager. The actual implementation has not been finished yet, nor has tests been written. Some code in the DefaultSqsBatchManager are just the functions copied over from the Sqs batch manager integration test.

Testing

Modifications to the core BatchManager are covered by tests in the last PR. No tests have been written yet for the SqsBatchManager but will be done in a separate PR covering implementation details.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)

Checklist

• I have read the CONTRIBUTING document
• Local run of mvn install succeeds
• My code follows the code style of this project
• My change requires a change to the Javadoc documentation
• I have updated the Javadoc documentation accordingly
• I have read the README document
• I have added tests to cover my changes
• All new and existing tests passed
• A short description of the change has been added to the CHANGELOG
• My change is to implement 1.11 parity feature and I have updated LaunchChangelog

License

• I confirm that this pull request can be released under the Apache 2 license

[zoewangg]

Build failed because of a checkstyle error in sqs, otherwise lgtm

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
25 Code Smells

80.9% Coverage
0.0% Duplication