petern48 commented on code in PR #24:
URL: https://github.com/apache/sedona-db/pull/24#discussion_r2411136954
##########
python/sedonadb/python/sedonadb/testing.py:
##########
@@ -302,6 +302,7 @@ def __init__(self):
import sedonadb
self.con = sedonadb.connect()
+ self.con.sql("SET datafusion.execution.target_partitions TO 1")
Review Comment:
> @Kontinuation so we probably should just put the single thread conf to
here
https://github.com/apache/sedona-db/blob/main/benchmarks/test_bench_base.py?
How about we add a helper function like `force_one_thread()` (I didn't think
too hard about the name) to SedonaDB and DuckDB engines so we can isolate this
behavior to the benchmarks. At the moment, this would slow down the SedonaDB
engine during the tests as well.
Or we add a parameter to the constructor
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
