Closed akharche closed 4 years ago
Hello @akharche! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
Performance results of DataFrame with mixed types of columns | Columns | Current compile time, s | New compile time, s | New/Current |
---|---|---|---|---|
16 | 11.092775 | 9.713415 | 0.875652 | |
32 | 28.611168 | 20.057998 | 0.701055 | |
64 | 128.307209 | 67.369364 | 0.525062 | |
128 | 803.671405 | 310.529887 | 0.386389 |
Performance results of DataFrame with mixed types of columns
Columns Current compile time, s New compile time, s New/Current 4 11.092775 9.713415 0.875652 8 28.611168 20.057998 0.701055 16 128.307209 67.369364 0.525062 32 803.671405 310.529887 0.386389
The results look very good.
@akharche examples failed
Extension for #801
Implementation of new DataFrame structure based on lists instead of tuples Improved df.count() codegen for testing Example:
Reproduce: