Skip to content

fix: Avro map deterministic encoding#587

Closed
radkawar wants to merge 3 commits into
apache:mainfrom
radkawar:fatstream-deterministic-map-encoding
Closed

fix: Avro map deterministic encoding#587
radkawar wants to merge 3 commits into
apache:mainfrom
radkawar:fatstream-deterministic-map-encoding

Conversation

@radkawar

Copy link
Copy Markdown

Update encode_internal function to sort map entries by key before encoding, ensuring deterministic output.

@Kriskras99

Copy link
Copy Markdown
Contributor

Hi @radkawar, why do you need this? This will severely degrade encoding performance, and the specification gives no guarantees on the key order.

@martin-g

Copy link
Copy Markdown
Member

I agree with @Kriskras99 that this is not a good idea.
It will even break interop with other SDKs.

@martin-g martin-g closed this Jul 14, 2026
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.

3 participants