-
Notifications
You must be signed in to change notification settings - Fork 13
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
'thrift_flume_event_server' not found #10
Comments
You seems correct. @cosmo0920 how about this? |
Yes, flume's thrift protocol definition is changed. |
Should I remove |
I thought the thrift protocol was updated couple of month ago here? so, that's outdated too? |
Removing |
I figured out you are using this thrift_source_protocol.rb, how about requiring this, would it work? |
No, it wouldn't. Currently, it does not work. Current thrift protocol sources cannot use new and older thrift protocol together in the same directory because their sources has same name which will cause conflict. To solve this issue, my idea is:
|
Will just using
Partially yes, but Flume thrift protocol had been updated over two years ago: apache/logging-flume@60da3d8#diff-cbeeb727e5da413a8c0abe58b0264d22 |
Basically, I am using the latest flume 1.6 and I cannot get this plugin to work. It is not clear from the readme, which version of flume this plugin is supporting. On the other hand, if I use this flume sink ( flume-ng-fluentd-sink ) and forward to 24224, I can completely skip using fluent-plugin-flume. So its a matter of using 1) flume thrift -> fluent-plugin-flume Apparently 1 is broken right now, and the plugin has issues requiring the rb files. I was trying to see if 1) could work, so I don't have to install additional jars in my flume installation in the case of 2) |
Your understanding is partially right, but latest Flume thrift protocol does not have sending flume event to outside flume functionality. |
It seems the current code removed 'thrift_flume_event_server' yet in in_flume.rb, it is still requiring the missing file. I think the thrift definition has changed?
The text was updated successfully, but these errors were encountered: