Open nbc opened 1 year ago
I ❤️ the idea @nbc!
The same logic with package withr
could be used to disallow missing values in partitioning variables in returned dataset?
I heart the idea @nbc! The same logic with package
withr
could be used to disallow missing values in partitioning variables in returned dataset?
There's two way to do it :
check the returned dataset object
We use the same method to check the returned object and warn/fail if it's not ok for us.
This way is more flexible as we can check if there's NA and the number of partition files for example.
really check the values of partioning columns for NA
We use something like this :
if (any(is.na(data[partitioning])))) cli_abort("bla bla bla")
I prefer the first way but we should perhaps wait for the talk to choose no ?
I will implement the first way anyway as soon as I've time.
Patch coverage: 96.66
% and project coverage change: -0.25
:warning:
Comparison is base (
84d4d57
) 99.71% compared to head (dd8eef2
) 99.47%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Hi @ddotta,
A first implementation to allow arguments checking.
We can do the same thing to check the returned dataset. It will be even simpler, the function will just take the dataset object.
Regards