Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Accept camel-case properties for producer configuration #148

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

emckissick
Copy link

Our property files use camelCase for a lot of our applications, and I
noticed that the KinesisProducerConfiguration fromProperties method is
doing a case sensitive search. This called us to change our applications properties specifically for Kinesis fields. Which is not ideal. Updated the method / property equality
check to ignore case.

Our property files use camelCase for a lot of our applications, and I
noticed that the KinesisProducerConfiguration fromProperties method is
doing a case sensitive search. Updated the method / property equality
check to ignore case.
@emckissick
Copy link
Author

If it makes sense to include other cases like snake__case and spinal-case, i'd be happy to update that in this PR.

@emckissick emckissick changed the title Use camel-case properties for producer configuration Accept camel-case properties for producer configuration Nov 2, 2017
…mel-case

# Conflicts:
#	java/amazon-kinesis-producer/src/test/java/com/amazonaws/services/kinesis/producer/KinesisProducerConfigurationTest.java
@yatins47 yatins47 closed this Nov 24, 2020
@yatins47 yatins47 deleted the branch awslabs:master November 24, 2020 15:19
@yatins47 yatins47 reopened this Nov 25, 2020
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.

2 participants