cloud-fan commented on code in PR #53578:
URL: https://github.com/apache/spark/pull/53578#discussion_r2646768033
##########
core/src/main/java/org/apache/spark/memory/TaskMemoryManager.java:
##########
@@ -364,7 +368,10 @@ public long pageSizeBytes() {
*
* @throws TooLargePageException
*/
- public MemoryBlock allocatePage(long size, MemoryConsumer consumer) {
+ public MemoryBlock allocatePage(
+ long size,
+ MemoryConsumer consumer,
+ boolean isRetry) {
Review Comment:
given we are touching this place, can we use `int retryCount` instead of a
boolean? And we can also log the retry count.
--
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]
