LLNL / ygm

Other
31 stars 22 forks source link

Add tests for parquet reader #191

Closed KIwabuchi closed 11 months ago

KIwabuchi commented 11 months ago
KIwabuchi commented 11 months ago

@steiltre Hey Trevor, could you review this PR when you have time?

steiltre commented 11 months ago

Looks good. This catches a lot more potential corner cases of the the new distribution of lines from Parquet files.