Replace multi_json and oj gems with standard Ruby json gem#2992
Conversation
|
💚 CLA has been signed |
8b60fef to
8158b11
Compare
|
This would be nice for another reason: currently, elasticsearch-ruby emits |
8158b11 to
f1e2762
Compare
|
It appears jruby 9.3 does not agree with the modern That said, jruby 9.3 has been EOL for two years (source). Maybe we can update CI? Would be glad to open a PR to remove 9.3/add newer jruby if that's something maintainers would be open to. PR opened: #3000.
|
f1e2762 to
8a72531
Compare
|
Let's get CI run and if it looks good merge #3000 then continue with this PR. |

Description
Proposing we remove the
multi_jsonandojgems in favour of the standard Rubyjsongem as it's now as fast or faster in the majority of cases compared tooj. Additionally allows us to simplify code quite a bit too. ✨Related efforts
jsongem for 4.0.0-beta.5 opensearch-project/opensearch-ruby#290ojand its initializer and addjsonto the Gemfile mastodon/mastodon#37752Implementation
More or less just a swap and cleanup. Nothing of particular note stands out.