mingmwang edited a comment on pull request #1924: URL: https://github.com/apache/arrow-datafusion/pull/1924#issuecomment-1064729039
> > Could you please guide me that who else can review this PR ? > > The current list of maintainers is listed at https://arrow.apache.org/committers/ -- ones who may be interested in this PR include @andygrove @Dandandan @rdettai @Jimexist @yjshen and @houqp > > > Sorry for pushing this hard, I need to close this the early the better so that I can continue the working on other parts. > > No problem -- I want to support you and you work as I think it is valuable. > > If you are able to break this PR into smaller pieces, as suggested on [#1924 (review)](https://github.com/apache/arrow-datafusion/pull/1924#pullrequestreview-901068370) I think you'll find sufficient review capacity to get it through. Hi, @alamb Ok. I will break this PR into three parts. 1. Rename from ExecutionContext to SessionContext and ExecutionContextState to SessionState, Add TaskContext and wrap the RuntimeEnv into TaskContext and pass down TaskContext into ExecutionPlan's execute() method, fix all the trivial UTs. 2. Changes related to the SessionContext and SessionContextState themselves, move the planner/optimizer from SessionConfig to SessionState. 3. Ballista related changes, add SessionContextRegistry, make SessionContext long running. SessionContext can created from BallistaConfig and can propagate to Executor. -- 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]
