Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21357
@tdas
The rationalization of this patch is to group functions which deal with
delta and snapshot files into one so that the difference between delta file and
snapshot file will be clearly shown (actually no difference other than allowing
TOMBSTONE value in delta file) as well as easy to document about these files.
It's also easier to add tests for delta / snapshot files.
Indeed my underlying rationalization is to make the class easier to
understand from newcomers (actually I found it helpful to group them logically
to understand the code better), but the file has been getting enough love from
various contributors so may not worth to put effort to make it easiler.
I respect the rule of Spark project, and happy to close if we don't feel
benefitial to go on. Let's close it and revisit some other one feels
benefitial. Thanks for providing your voice on this!
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
