david-streamlio commented on PR #77:
URL: https://github.com/apache/pulsar-connectors/pull/77#issuecomment-4936232906
Rebased on master now that #79 has merged. The dependency is confirmed
resolved, not assumed:
Before #79, this test failed with the very bug it guards:
```
DynamoDBSourceIntegrationTest > testReadFromDynamoDBStream FAILED
java.lang.IllegalStateException: Only one of Region or
EndpointConfiguration may be set.
```
Against current master:
```
DynamoDBSourceIntegrationTest > testReadFromDynamoDBStream PASSED
BUILD SUCCESSFUL in 50s
```
The full `:dynamodb:test` suite is green — the LocalStack integration test
plus #79's four `DynamoDBSourceConfigClientBuilderTest` unit tests. Diff is
still exactly two files, both tests. Ready for review.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
