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.

Sign up for GitHub

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

Jump to bottom

remove ruby execution #11236

Closed
1 task
jsvd opened this issue Oct 17, 2019 · 0 comments · Fixed by logstash-plugins/logstash-input-redis#84, elastic/logstash-devutils#89, #12517, logstash-plugins/logstash-integration-rabbitmq#39 or logstash-plugins/logstash-mixin-rabbitmq_connection#64
Closed
1 task

remove ruby execution #11236

jsvd opened this issue Oct 17, 2019 · 0 comments · Fixed by logstash-plugins/logstash-input-redis#84, elastic/logstash-devutils#89, #12517, logstash-plugins/logstash-integration-rabbitmq#39 or logstash-plugins/logstash-mixin-rabbitmq_connection#64
Labels
breaking change v8.0.0

Comments

@jsvd
Copy link
Member

jsvd commented Oct 17, 2019 •
edited by yaauie
Loading

With java execution we no longer need the ruby one. There are know issues blocking this removal:

The Java Execution Engine has been the default engine since Logstash 7.0, and is capable of using plugins written in either Ruby or Java; removing the Ruby Execution Engine will not affect our ability to run existing pipelines, but will greatly reduce the surface area of all features and fixes within the engine(s), since we will no longer need to target two engines simultaneously.
@roaksoax roaksoax mentioned this issue May 27, 2020
Closed
35 tasks
This was referenced Dec 1, 2020
Closed
Merged
Merged
Merged
Merged
Merged
@jsvd jsvd reopened this Dec 14, 2020
kaisecheng added a commit that referenced this issue Dec 14, 2020
@kaisecheng
Remove ruby pipeline (#12496)
5239fc9 
* remove Logstash::Pipeline
* remove Engine.Ruby
* rewrite test cases which depend on ruby pipeline
Fixed: #11236
kaisecheng added a commit that referenced this issue Dec 14, 2020
@kaisecheng
remove Logstash::SignalEvent (#12505)
0f4e36a 
Fixed: #11236
kaisecheng added a commit that referenced this issue Dec 14, 2020
@kaisecheng
Remove ruby execution engine config (#12506)
185e142 
* remove config pipeline.java_execution
* remove ruby engine doc
Fixed: #11236
@kaisecheng kaisecheng mentioned this issue Dec 14, 2020
Merged
14 tasks
kaisecheng added a commit that referenced this issue Dec 15, 2020
@kaisecheng
Remove ruby execution engine (#12517)
4bc9dad 
* remove Logstash::Pipeline
* remove logstash/event, Logstash::SignalEvent
* remove Engine.Ruby on java side
* remove config pipeline.java_execution
Fixed: #11236
@kaisecheng kaisecheng mentioned this issue Dec 21, 2020
Merged
kaisecheng added a commit to kaisecheng/logstash that referenced this issue Dec 21, 2020
@kaisecheng
add compatibility for plugins not in support matrix
94fcb81 
Fixed: elastic#11236
@kaisecheng kaisecheng mentioned this issue Dec 21, 2020
Merged
yaauie added a commit to yaauie/logstash that referenced this issue May 20, 2021
@yaauie
doc: add REE removal to 8.0 breaking changes
724699d 
Related: elastic#11236
@yaauie yaauie mentioned this issue May 20, 2021
Open
1 task
yaauie pushed a commit that referenced this issue Jan 25, 2022
@kaisecheng
add compatibility for plugins not in support matrix (#12534)
6a28ac1 
Fixed: #11236
yaauie pushed a commit to yaauie/logstash that referenced this issue Jan 25, 2022
@kaisecheng @yaauie
add compatibility for plugins not in support matrix (elastic#12534)
8cab7d4 
Fixed: elastic#11236
(cherry picked from commit 6a28ac1)
yaauie added a commit that referenced this issue Jan 25, 2022
@yaauie @kaisecheng
add compatibility for plugins not in support matrix (#12534) (#13675)
faa436d 
Fixed: #11236
(cherry picked from commit 6a28ac1)

Co-authored-by: kaisecheng <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking change v8.0.0
Projects
None yet
Milestone
No milestone
1 participant
@jsvd