Skip to content

The MapVisitor didn't handle correctly Uint and Uin64#15187

Merged
michalpristas merged 1 commit intoelastic:fleetfrom
ph:agent/fix-casting-uint-issue
Dec 18, 2019
Merged

The MapVisitor didn't handle correctly Uint and Uin64#15187
michalpristas merged 1 commit intoelastic:fleetfrom
ph:agent/fix-casting-uint-issue

Conversation

@ph
Copy link
Copy Markdown
Contributor

@ph ph commented Dec 18, 2019

This PR add support for these types to make sure the convertion is
correctly done. The problem was the generated map had NIL values instead
of having the unsigned int values.

Fix: #15182

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ingest (Project:fleet)

Copy link
Copy Markdown
Contributor

@michalpristas michalpristas left a comment

Choose a reason for hiding this comment

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

LGTM

This PR add support for these types to make sure the convertion is
correctly done. The problem was the generated map had NIL values instead
of having the unsigned int values.

Fix: #15182
@michalpristas michalpristas merged commit 4b479d2 into elastic:fleet Dec 18, 2019
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
This PR add support for these types to make sure the convertion is
correctly done. The problem was the generated map had NIL values instead
of having the unsigned int values.

Fix: elastic#15182
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants