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

[BUG] Retract message for Aggregate table will lead to NPE for record ttl compaction #4821

Open
2 tasks done
xiangyuf opened this issue Jan 2, 2025 · 0 comments
Open
2 tasks done
Labels
enhancement New feature or request

Comments

@xiangyuf
Copy link
Contributor

xiangyuf commented Jan 2, 2025

Search before asking

  • I searched in the issues and found nothing similar.

Motivation

For an aggregation Paimon PK table, if it receives a retract message at first, it will inevitably generate a +I message with a negative value for SumAggField or null value for other agg fields. If this table has enabled record level ttl, this will throw an IllegalArgumentException during compaction if the ttl time field is null.

The first react message may generated from the out-of-order issue during now-primary key hashing in streaming computation.

image

Solution

Ignore first retract message for aggregate table if its record level TTL is enabled. In most cases, the first retract message is meaningless for an aggregation table.

Anything else?

No response

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@xiangyuf xiangyuf added the enhancement New feature or request label Jan 2, 2025
@xiangyuf xiangyuf changed the title [Feature] Ignore first retract message for AggregateMergeFunction when record level TTL is enabled [BUG] Retract message for Aggregate table will lead to NPE for record ttl compaction Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant