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.

Sign up for GitHub

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

Jump to bottom

Accept camel-case properties for producer configuration #148

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

Accept camel-case properties for producer configuration #148

wants to merge 4 commits into from

Commits on Nov 1, 2017

  1. Use camel-case properties for producer configuration 

    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
    emckissick committed Nov 1, 2017
    Configuration menu
    Copy the full SHA
    6e3f30d View commit details
    Browse the repository at this point in the history

  2. Make the test actually use camel-case.

    @emckissick
    emckissick committed Nov 1, 2017
    Configuration menu
    Copy the full SHA
    e58fbac View commit details
    Browse the repository at this point in the history

Commits on Nov 13, 2017

  1. Merge remote-tracking branch 'upstream/master' into properties-use-ca… 

    …mel-case

# Conflicts:
#	java/amazon-kinesis-producer/src/test/java/com/amazonaws/services/kinesis/producer/KinesisProducerConfigurationTest.java
    @emckissick
    emckissick committed Nov 13, 2017
    Configuration menu
    Copy the full SHA
    a2c9ef9 View commit details
    Browse the repository at this point in the history

  2. Update to stay in sync with new merge.

    @emckissick
    emckissick committed Nov 13, 2017
    Configuration menu
    Copy the full SHA
    2b8fb04 View commit details
    Browse the repository at this point in the history