[GitHub] [doris] Kikyou1997 commented on pull request #14733: [ehancement](nereids) Read lock table when analyze and generate query plan

[GitBox <gi...@apache.org>]

Kikyou1997 commented on PR #14733:
URL: https://github.com/apache/doris/pull/14733#issuecomment-1342652131

   > I think table read lock should span the statement's planner lifecycle, the view and subquery maybe reentrant the same table lock. And you should use the `Stack<ReentrantReadWriteLock>` in the StatementContext to save the lock order and then we can release by the reverse order after exit the planner, catch Throwable should be considered. And you should try require read lock with timeout for prevent dead lock.
   
   Done, thanks for these suggestions


-- 
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: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org