Skip to content

fix: restore separate BigDecimal routing to avoid JRuby panic - #71

Merged
yaauie merged 2 commits into
logstash-plugins:mainfrom
yaauie:bigdecimal-timestamp-support
Aug 22, 2022
Merged

yaauie merged 2 commits into
logstash-plugins:mainfrom
yaauie:bigdecimal-timestamp-support

Conversation

@yaauie

@yaauie yaauie commented Aug 20, 2022

Copy link
Copy Markdown
Contributor

Adds a test case for large exponent-notation timestamp fields and restores the separate routing of BigDecimal-decoded timestamps.

Relates: #67
Resolves: #70

Comment thread CHANGELOG.md Outdated

@andsel andsel left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yaauie
yaauie merged commit eafb220 into logstash-plugins:main Aug 22, 2022
@yaauie
yaauie deleted the bigdecimal-timestamp-support branch August 22, 2022 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Negative exponent panic

2 participants