Description of changes: Moving the shutdownNotification.shutdownComplete() call into the ShardConsumer.shutdownComplete method.
Having the shutdownNotification in ShutdownTask.java breaks backwards compatibility since the constructor is being modified. For future reference, any public method should use the @Builder annotation since it's better for backwards compatibility.
See PR #1302 for the original commits.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Issue #, if available: N/A
Description of changes: Moving the
shutdownNotification.shutdownComplete()call into theShardConsumer.shutdownCompletemethod.Having the shutdownNotification in
ShutdownTask.javabreaks backwards compatibility since the constructor is being modified. For future reference, any public method should use the@Builderannotation since it's better for backwards compatibility.See PR #1302 for the original commits.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.