Refactor and improve $scan/read_parquet()

[sorhawell]

No description provided.

[sorhawell]

@eitsupi sorry for circling around the match.call eval.

I noticed in new utest "scan read parquet from other process" that replacement of eval.parent with eval will not work. But using eval.parent makes the frame stack a bit strange, and parent frame could very well be ´R_GlobalEnv´. Maybe that could give some unexpected behavior one day.

How about do.call(f, args)?

pl$read_parquet(x,y,z) { 
  args = as.list(environment())
    result({
      do.call(pl$scan_parquet, args)$collect()
    }) |>
      unwrap("in pl$read_parquet(): ")
  }

[etiennebacher]

How about do.call(f, args)?

This looks more robust indeed

[eitsupi]

How about do.call(f, args)?

Actually I am not familiar with these but they seem to work well, thanks!

[etiennebacher]

Thanks! I'll just tweak a few things before merging