wsry commented on pull request #18173: URL: https://github.com/apache/flink/pull/18173#issuecomment-1003059833
> I guess in LocalBufferPool we can have four options: > I. Preemptively acquire factoryLock (2.) in methods that are used rarely > II. Decouple logic in two steps. First do something under the lock 1., release lock 1., as second do something that requires lock 2. > III. First acquire lock 1., but if it turns out that we will need to acquire lock 2., return from the NetworkBufferPool back to the LocalBufferPool, release lock 1., acquire lock 2. and then acquire lock 1. again. > IV. Never acquire lock 2. @pnowojski I think the local buffer pool is already doing these, though some cases does not exist, so we need do nothing? -- 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]
