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

Kinesis Elasticsearch Sink: configure transport port #2102

Closed
xkidro opened this issue Oct 5, 2015 · 5 comments
Closed

Kinesis Elasticsearch Sink: configure transport port #2102

xkidro opened this issue Oct 5, 2015 · 5 comments
Assignees

Comments

@xkidro
Copy link

xkidro commented Oct 5, 2015

I searched the source code from https://github.com/snowplow/snowplow/tree/master/4-storage/kinesis-elasticsearch-sink I can't seem to find where the 9300 is hard coded and can't find a way to configure it as well.

@alexanderdean alexanderdean added this to the Kinesis #8 milestone Oct 5, 2015
@fblundun
Copy link
Contributor

fblundun commented Oct 5, 2015

Hi @xkidro ,

9300 is the default value used in the amazon-kinesis-connectors library. We override several of these default values here, but the port is not (yet) one of them.

@xkidro
Copy link
Author

xkidro commented Oct 6, 2015

Hello @fblundun thank you for the clarification, I have a question, it's not really related with this but it is related to the other issue I posted, didn't really study these but would it be possible for the sink to use the http api instead of the transport client? Does the kinesis connectors library support this? If so, would it be hard to modify the sink to work with that? I really want to use the new AWS elasticsearch service but it doesn't support the transport client :(

@fblundun
Copy link
Contributor

fblundun commented Oct 7, 2015

Unfortunately not - we only support the transport client. Compatibility with Amazon Elasticsearch Service would be nice but as you say they don't support the transport client (source). A pull request adding the option to use the HTTP client would be welcome!

@fblundun
Copy link
Contributor

fblundun commented Oct 7, 2015

Note that we don't actually use the amazon-kinesis-connectors Elasticsearch emitter - we implement that ourselves:

@fblundun fblundun modified the milestones: Kinesis near future, Kinesis #8 Mar 22, 2016
@alexanderdean alexanderdean changed the title Scala Elasticsearch sink configure transport port ? Kinesis Elasticsearch: configure transport port Mar 24, 2016
@alexanderdean alexanderdean modified the milestones: Release 83 [KIN] Bird TBC, Kinesis near future Jul 18, 2016
@alexanderdean
Copy link
Member

Bringing forwards as pretty straightforward add

@alexanderdean alexanderdean changed the title Kinesis Elasticsearch: configure transport port Kinesis Elasticsearch Sink: configure transport port Jul 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants