Serialize engine config in new pdsh benchmark CLI#22365
Serialize engine config in new pdsh benchmark CLI#22365TomAugspurger wants to merge 10 commits intorapidsai:mainfrom
Conversation
This updates the cudf-polars benchmarks CLI to serialize the engine configuration. This will let us see what options were *actually* used.
|
Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually. Contributors can view more details about this message here. |
This adds back a `--spill-to-pinned-memory` CLI option and threads it through for the new frontends.
|
This needs more work. We need to ensure that the |
|
I've updated this to serialize And here are some example values of Unfortunately, I don't think rapidsmpf currently has a way to view the actual values the options resolve to without explicitly setting or using them, which is why the dask/ray examples only have the |
|
@madsbk do you have any ideas about how to handle #22365 (comment)? Specifically,
|
I don’t think it is possible with the current design, where each call site parses the string value and defines its own default.
An empty string means that no one has accessed and parsed that option yet. |
|
OK, I think this might be all we can do on the cudf-polars side then. I've opened rapidsai/rapidsmpf#1014 to track what I think needs to happen on the rapidsmpf side, and then hopefully we can get back to a state where all of the actual runtime options are being recorded. For now, I'll merge this so that the non-rapidsmpf options are being recored properly. |
Description
This updates the cudf-polars benchmarks CLI to serialize the engine configuration. This will let us see what options were actually used.