delta-io / kafka-delta-ingest

A highly efficient daemon for streaming data from Kafka into Delta Lake
Apache License 2.0
359 stars 79 forks source link

Refactor writer::apply_null_counts to reduce nesting #113

Closed xianwill closed 2 years ago

xianwill commented 2 years ago

This PR contains a small code cleanup of writer::apply_null_counts to reduce nesting and improve readability.

@thovoll and I paired on this together (FUN!) as part of what will hopefully become a semi-regular code cleanup pairing session. We noted that there is some potential for additional cleanup in these two functions, but just this small change as-is represents a worthwhile well-scoped and safe incremental improvement.