fix: fix mutation types in delete DAG

[fuziontech]

Problem

simple fix for types that break at runtime

Changes

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

Does this work well for both Cloud and self-hosted?

How did you test this code?

[greptile-apps]

PR Summary

This PR fixes type mismatches in the person deletion DAG (Directed Acyclic Graph) implementation, specifically in the mutation handling logic.

• Fixed type annotation in wait_for_delete_mutations from Mutation to ShardMutations (Dict[int, Mutation])
• Updated implementation to properly iterate over all shard mutations instead of handling a single mutation
• Added proper handling of empty mutation cases in delete_person_events
• Aligned mutation handling with the actual multi-shard architecture of ClickHouse
• Improved type safety while maintaining backward compatibility with existing deletion verification process

_{1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}