-
Notifications
You must be signed in to change notification settings - Fork 44
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
[KARAF-5454] - Collector socket - Add UDP protocol support #18
Conversation
Thanks. I gonna take a look. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also please update the documentation and the default cfg file ?
|
||
# Protocol tcp(default) or udp | ||
#protocol=tcp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please avoid to do useless changes (it's just noise in the commit ;)).
Any update about this PR ? |
Sorry, I check this |
PR review fix : clean noise at the end of the default config file PR review add : update documentation
retest this please |
FAILURE --none-- |
Failure doesn't seem related as it's on the JMS collector. Launching a new build to verify. |
retest this please |
SUCCESS --none-- |
Build is now OK. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks. I gonna squash and merge.
Improve the collector-socket adding compatibility with the UDP protocol.
Use a DatagramServer instead of a ServerSocket for UDP.
Changes :
Add a new properties in the collector cfg file to define the protocol :
#Protocol tcp(default) or udp
#protocol=tcp
Instanciate a DatagramServer in the case of udp protocol is define in the cfg
class
org.apache.karaf.decanter.collector.socket.SocketCollector