Skip to content
This repository was archived by the owner on Sep 2, 2025. It is now read-only.

Added new high performance HTTP Event Collector (HEC) output adaptor.#3

Closed
damiendallimore wants to merge 1 commit into
spring-attic:masterfrom
damiendallimore:INTEXT-201
Closed

Added new high performance HTTP Event Collector (HEC) output adaptor.#3
damiendallimore wants to merge 1 commit into
spring-attic:masterfrom
damiendallimore:INTEXT-201

Conversation

@damiendallimore

Copy link
Copy Markdown

With the recent release of Splunk version 6.3 we now have a new high performance data collector called the HTTP Event Collector (HEC), http://blogs.splunk.com/2015/10/06/http-event-collector-your-direct-event-pipe-to-splunk-6-3/

This should now be used over our legacy data collectors which have been available as output adaptors in Spring Integration for some time.

This pull request contains a new output integration adaptor for sending events to Splunk via HEC.

I also refactored the SplunkEvent class which was riddled with legacy logic that is now irrelevant.I also updated the output of the SplunkEvent class to produce simple JSON which is more optimal that KV pairs for Splunk.

Damien Dallimore (Developer Evangelist @ Splunk , ddallimore@splunk.com)

I have signed and agree to the terms of the SpringSource Individual
Contributor License Agreement.

Refactored SplunkEvent class to produce simple JSON event.
@artembilan

Copy link
Copy Markdown
Contributor

Hello, @damiendallimore !

Sorry for delay. We were enough busy with releases for SI core project and other related.
So, now I can switch to your work for review.

Comment thread gradle.properties

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor, of course, but that would be better to configure the IDE to add an new line in the end of file.
At least, to avoid such a comments in the future 😄

@artembilan

Copy link
Copy Markdown
Contributor

So, from big height that's all.
There are a lot to do yet, so let me know your point.

I'd prefer do not do so much re-work myself, because it is your PR, but if you don't have enough time to bring it into mind, that works for us, too.

Thank you for the contribution one more time!

@artembilan artembilan mentioned this pull request Dec 10, 2015
@artembilan

Copy link
Copy Markdown
Contributor

Closed in favor of #5 .
Sorry @damiendallimore for dropping your PR, but your work is appreciated anyway.

Thank you for the contribution and looking forward for more!

@artembilan artembilan closed this Dec 10, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants