Update EventHelper.java #17

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@martinschaef

Avoid null pointer exception in line 71 for the case "if (event == null || event.getBody() == null)" in line 44/45

Update EventHelper.java
Avoid null pointer exception in line 71 for the case "if (event == null || event.getBody() == null)" in line 44/45
@harishreedharan

This comment has been minimized.

Show comment
Hide comment
@harishreedharan

harishreedharan Apr 21, 2015

Contributor

Flume does not accept PRs. Could you file a jira here: https://issues.apache.org/jira/browse/FLUME and submit this as a patch? Thanks!

Contributor

harishreedharan commented Apr 21, 2015

Flume does not accept PRs. Could you file a jira here: https://issues.apache.org/jira/browse/FLUME and submit this as a patch? Thanks!

@martinschaef

This comment has been minimized.

Show comment
Hide comment
@martinschaef

martinschaef Apr 21, 2015

I'm not familiar with jira ... I guess I created an issue and not a patch. But its a one-liner anyway.

I'm not familiar with jira ... I guess I created an issue and not a patch. But its a one-liner anyway.

hejiang2000 added a commit to hejiang2000/flume that referenced this pull request Jun 19, 2018

完成 PassiveHttpSink 部分 (#17)
* initial

* +

* =

* +

* *

* +

* PassiveHttpSink passed unit tests

* 支持 内容头字符,尾字符,事件分隔符。

* format style

hejiang2000 added a commit to hejiang2000/flume that referenced this pull request Jun 19, 2018

完成 PassiveHttpSink 部分 (#17) (#18)
* initial

* +

* =

* +

* *

* +

* PassiveHttpSink passed unit tests

* 支持 内容头字符,尾字符,事件分隔符。

* format style
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment