ARROW-15016: [R] `show_exec_plan` for an `arrow_dplyr_query` - #13541

Merged

paleolimbot merged 82 commits into

apache:masterfrom

dragosmg:show_query

Jul 22, 2022

Commits on Jul 11, 2022

add a `_ToString` method for the ExecPlan
dragosmg
committed
403d9df
add `ToString` method for the `ExecPlan` R6 object
dragosmg
committed
4a1f94e
define `show_arrow_query()` and add it to `print.arrow_dplyr_query()`
dragosmg
committed
f03ced0
rename `show_arrow_query()` to `show_exec_plan()` + unit tests
dragosmg
committed
fdd20be
lint
dragosmg
committed
350bea3
use `expect_snapshot()` instead of `expect_output()`
dragosmg
committed
c889ec0
remove empty row
dragosmg
committed
1260540
update test and snapshot
dragosmg
committed
1f4801d
document + export `show_exec_plan()`
dragosmg
committed
254e14c
example + redocument
dragosmg
committed
6294765
add `show_exec_plan` to r/_pkgdown.yml under *Computation*
dragosmg
committed
cb5d5b7
bump ci
dragosmg
committed
424f12a

Commits on Jul 12, 2022

run example only when {dplyr} is available
dragosmg
committed
ccd8f76
remove snapshots and test with `expect_output()`
dragosmg
committed
83e5492
lint
dragosmg
committed
b9c0c30
bump ci
dragosmg
committed
07de250
docs
dragosmg
committed
0e3b894

Commits on Jul 13, 2022

improved `show_exec_plan()` docs
dragosmg
committed
3aab4e0
Merge branch 'master' into show_query
dragosmg
authored
3355135
use `adq`
dragosmg
committed
b9b4d49
add minimal test
dragosmg
committed
4779f08
failing unit tests for `arrange()` and `head()` "nodes"
dragosmg
committed
5730cec
add `show_query.arrow-dplyr_query()` and `explain.arrow_dplyr_query()`
dragosmg
committed
23f5f22
tests for `show_query()` & `explain()` + clarifications
dragosmg
committed
2eb3cdb
move dataset tests to test-dataset-dplyr.R + unskip
dragosmg
committed
2b33495
clean-up
dragosmg
committed
4ff3065
clean-up
dragosmg
committed
d686f9d

Commits on Jul 18, 2022

define `ExecPlan_ToStringWithSink` C++ function
dragosmg
committed
d452a40
define the `ToStringWithSink()` R6 ExecPlan method and use it in `show_exec_plan()`
dragosmg
committed
f4a1ba4
updated unit tests
dragosmg
committed
ac7d756
comments
dragosmg
committed
761b66d
clang style
dragosmg
committed
a36f826
comment
dragosmg
committed
d887d42
clang style
dragosmg
committed
effc963
clang style
dragosmg
committed
4571df4
clang style
dragosmg
committed
d1e4ce7
update dataset tests
dragosmg
committed
e2220fe
comment
dragosmg
committed
3c96fb7
bump ci
dragosmg
committed
068dc6d
don't run the cyclocomp linter on the ExecPlan R6 definition
dragosmg
committed
fb2a5a0

Commits on Jul 19, 2022

typo
dragosmg
committed
c14a497
`&&`
dragosmg
committed
aac9edc
rename `ToStrinWithSink` to `ToString` and keep only this method
dragosmg
committed
3faecc6
docs
dragosmg
committed
1afcce9
C++ lint
dragosmg
committed
1de3447
add `ExecPlan_prepare` helper and update `ExecPlan_ToString` C++ functions
dragosmg
committed
f40e5d2
update ToString R6 ExecPlan method
dragosmg
committed
8aae3c3
c++ lint
dragosmg
committed
fae74ae
C++ lint
dragosmg
committed
ebd4a02
another C++ lint
dragosmg
committed
311b70b
simpler ExecPlan_prepare
dragosmg
committed
d0bc159
C++ lint
dragosmg
committed
d7834b0
more C++ linting
dragosmg
committed
9a02651
extend ExecPlan_prepare and use it for ExecPlan_run
dragosmg
committed
ac37a9d
C++ lint
dragosmg
committed
238e406
C++ lint
dragosmg
committed
9a34255
include safe call into R header file
dragosmg
committed
1db36b6
revert to old ExecPlan_run
dragosmg
committed
2153451
c++ lint
dragosmg
committed
807aa88
simplified ExecPlan_prepare
dragosmg
committed
6141c45

Commits on Jul 20, 2022

`ExecPlan_prepare` to return both the plan and the sink node
dragosmg
committed
9af1403
C++ lint
dragosmg
committed
1f88ff7
fix the rabbit hole that I led Dragos down
paleolimbot
committed
fbb4c1e
always call startproducing
paleolimbot
committed
da73019

Commits on Jul 21, 2022

bump ci
dragosmg
committed
ee90354
updated unit tests + snapshots
dragosmg
committed
66dcd22
updated dataset unit tests + snapshots
dragosmg
committed
0482c6c
remove `BuildAndShow` + add `ToString`
dragosmg
committed
ad5024f
update `as_record_batch_reader` and use it inside `show_exec_plan()`
dragosmg
committed
20799f9
docs
dragosmg
committed
21bc8c3
lints
dragosmg
committed
6376bb2

Commits on Jul 22, 2022

revert to da730196fc62b7a1572dc2d40e2550744039a142
dragosmg
committed
5751543
don't start producing the plan
dragosmg
committed
e29a835
remove separate tests for `show_query()` and `explain()`
dragosmg
committed
1309e0f
revert to fb2a5a00fc24d16da8e0a596256e0c18b02f5289 and use the `BuildAndShow` name
dragosmg
committed
5f890dc
docs + comments to indicate the duplicate code in:
dragosmg
committed
c4912a4
removed `ungroup()` call
dragosmg
committed
5aaa3cd
clang lint
dragosmg
committed
ce8b17e
comment
dragosmg
committed
0ffa30a
warn and don't build & print the ExecPlan when we have a nested query
dragosmg
committed
6a8c753
Merge branch 'master' into show_query
dragosmg
authored
7fc20ca
comments in `ExecPlan_prepare`
dragosmg
committed
35e9ef8