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.

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

[LI-HOTFIX] KAFKA-9855 - return cached Structs for Schemas with no fields #77

Merged
merged 1 commit into from Apr 15, 2020

Conversation

radai-rosenblatt
Copy link

TICKET =
LI_DESCERIPTION = committing to our fork until upstream picks this up
EXIT_CRITERIA = when KAFKA-9855 is accepted upstream (specifically - apache#8472)

Signed-off-by: radai-rosenblatt radai.rosenblatt@gmail.com

…elds

TICKET =
LI_DESCERIPTION = committing to our fork until upstream picks this up
EXIT_CRITERIA = when KAFKA-9855 is accepted upstream (specifically - apache#8472)

Signed-off-by: radai-rosenblatt <radai.rosenblatt@gmail.com>
Copy link

@gitlw gitlw left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Copy link

@smccauliff smccauliff left a comment

Choose a reason for hiding this comment

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

So this is like Collections.emptyList() but for Structs.

@radai-rosenblatt
Copy link
Author

radai-rosenblatt commented Apr 13, 2020

there's a higher-hanging fruit of Structs where values are all primitives. in theory we could specialize a Struct to have values = int[] instead of Object[] and save a significant chunk of memory on wrappers, but i'll punt on that for later.

@radai-rosenblatt radai-rosenblatt merged commit c70c676 into linkedin:2.3-li-1 Apr 15, 2020
gitlw pushed a commit that referenced this pull request Jun 13, 2020
…elds (#77)

TICKET =
LI_DESCERIPTION = committing to our fork until upstream picks this up
EXIT_CRITERIA = when KAFKA-9855 is accepted upstream (specifically - apache#8472)

Signed-off-by: radai-rosenblatt <radai.rosenblatt@gmail.com>
gitlw pushed a commit that referenced this pull request Jun 13, 2020
…elds (#77)

TICKET =
LI_DESCERIPTION = committing to our fork until upstream picks this up
EXIT_CRITERIA = when KAFKA-9855 is accepted upstream (specifically - apache#8472)

Signed-off-by: radai-rosenblatt <radai.rosenblatt@gmail.com>
gitlw pushed a commit that referenced this pull request Jun 17, 2020
…elds (#77)

TICKET =
LI_DESCERIPTION = committing to our fork until upstream picks this up
EXIT_CRITERIA = when KAFKA-9855 is accepted upstream (specifically - apache#8472)

Signed-off-by: radai-rosenblatt <radai.rosenblatt@gmail.com>
gitlw pushed a commit that referenced this pull request Jun 17, 2020
…elds (#77)

TICKET =
LI_DESCERIPTION = committing to our fork until upstream picks this up
EXIT_CRITERIA = when KAFKA-9855 is accepted upstream (specifically - apache#8472)

Signed-off-by: radai-rosenblatt <radai.rosenblatt@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants