[GitHub] [flink] reswqa commented on pull request #22447: [FLINK-31764][runtime] Get rid of numberOfRequestedOverdraftMemorySegments in LocalBufferPool

2023-05-05 Thread via GitHub
reswqa commented on PR #22447: URL: https://github.com/apache/flink/pull/22447#issuecomment-1536997003 CI passed, merging... -- 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.

[GitHub] [flink] reswqa commented on pull request #22447: [FLINK-31764][runtime] Get rid of numberOfRequestedOverdraftMemorySegments in LocalBufferPool

2023-05-04 Thread via GitHub
reswqa commented on PR #22447: URL: https://github.com/apache/flink/pull/22447#issuecomment-1535639126 Rebased on master to avoid the CI problem caused by [FLINK-30972](https://issues.apache.org/jira/browse/FLINK-30972). -- This is an automated message from the Apache Git Service. To resp

[GitHub] [flink] reswqa commented on pull request #22447: [FLINK-31764][runtime] Get rid of numberOfRequestedOverdraftMemorySegments in LocalBufferPool

2023-05-04 Thread via GitHub
reswqa commented on PR #22447: URL: https://github.com/apache/flink/pull/22447#issuecomment-1535049629 @flinkbot run azure -- 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. T

[GitHub] [flink] reswqa commented on pull request #22447: [FLINK-31764][runtime] Get rid of numberOfRequestedOverdraftMemorySegments in LocalBufferPool

2023-05-04 Thread via GitHub
reswqa commented on PR #22447: URL: https://github.com/apache/flink/pull/22447#issuecomment-1534797190 Squashed the fix-up commit, let's waiting for the CI green. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [flink] reswqa commented on pull request #22447: [FLINK-31764][runtime] Get rid of numberOfRequestedOverdraftMemorySegments in LocalBufferPool

2023-05-04 Thread via GitHub
reswqa commented on PR #22447: URL: https://github.com/apache/flink/pull/22447#issuecomment-1534754347 Thanks @akalash for carefully confirming this change! I have been extracted the logic related to request buffer from global pool and increase the `numberOfRequestedMemorySegments` into a c

[GitHub] [flink] reswqa commented on pull request #22447: [FLINK-31764][runtime] Get rid of numberOfRequestedOverdraftMemorySegments in LocalBufferPool

2023-04-24 Thread via GitHub
reswqa commented on PR #22447: URL: https://github.com/apache/flink/pull/22447#issuecomment-1521183716 Thanks @1996fanrui for the quick review, I have updated this in a fix-up commit, please take a look~ -- This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [flink] reswqa commented on pull request #22447: [FLINK-31764][runtime] Get rid of numberOfRequestedOverdraftMemorySegments in LocalBufferPool

2023-04-22 Thread via GitHub
reswqa commented on PR #22447: URL: https://github.com/apache/flink/pull/22447#issuecomment-1518950603 Hi @1996fanrui and @akalash, would you mind taking a look at this? Thx~ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and