andygrove commented on code in PR #1485:
URL: https://github.com/apache/datafusion-comet/pull/1485#discussion_r1989931497


##########
spark/src/main/java/org/apache/spark/shuffle/comet/CometBoundedShuffleMemoryAllocator.java:
##########
@@ -48,7 +48,7 @@
  * Thus, this allocator is used to allocate separate off-heap memory 
allocation for Comet JVM
  * shuffle and execution apart from Spark's on-heap memory configuration.
  */
-public final class CometTestShuffleMemoryAllocator extends 
CometShuffleMemoryAllocatorTrait {
+public final class CometBoundedShuffleMemoryAllocator extends 
CometShuffleMemoryAllocatorTrait {

Review Comment:
   I like the new name 👍 



-- 
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: github-unsubscr...@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to