kbendick edited a comment on pull request #2803:
URL: https://github.com/apache/iceberg/pull/2803#issuecomment-877896139


   Thank you for your submission @southernriver! 
   
   This seems like a somewhat large behavior change. Can you open a GitHub 
issue to track this / for general discussion? Or possibly this has been 
discussed on the mailing list (in which case possibly you can open an issue and 
link to the dev list discussion)? Also, an example query / situation you 
encounter this in could be added there and that would help greatly. I might be 
thinking of something else, but I thought that at a certain point, planning is 
distributed once a certain heuristic is passed.
   
   One thing I'd like to see discussed is the abilitu to opt-into this 
behavior. I'd be more comfortaenwitj this change if we could (at least 
initially) allow users to opt into or out of using the thread pool to do the 
planning. I imagine there are scenarios in which it's a detriment. An issue 
(and then either the dev list or that issue) would be a much better place to 
discuss that in my opinion.
   
   I’m not necessarily adverse to this change, but I do think having an issue 
to track a relatively large change in functionality (single threaded to 
multithreaded) would be nice as this is a pretty large change in behavior with 
no way to really opt out (or any documentation on the new table property).
   
   Also, if you could more completely describe what the query is that you're 
encountering these times and what your dataset looks like (ideally in the 
issue), that would be great. It's possible that with so many files, OPTIMIZE 
and other table maintenance tasks need to be fun.


-- 
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]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to