(celeborn) branch main updated: [MINOR] Fix PbSerDeUtilsTest failure

[nicholasjiang]

This is an automated email from the ASF dual-hosted git repository.

SteNicholas pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/celeborn.git


The following commit(s) were added to refs/heads/main by this push:
     new b131a400a [MINOR] Fix PbSerDeUtilsTest failure
b131a400a is described below

commit b131a400a239486fe36c659ed6b4397a27272a82
Author: Sanskar Modi <sanskarmod...@gmail.com>
AuthorDate: Tue Jun 9 03:31:17 2026 +0800

    [MINOR] Fix PbSerDeUtilsTest failure
    
    ### What changes were proposed in this pull request?
    
    Fix PbSerDeUtilsTest failure for spark4 jobs
    
    ```
    Error:  
/home/runner/work/celeborn/celeborn/common/src/test/scala/org/apache/celeborn/common/util/PbSerDeUtilsTest.scala:851:
 comparing values of types Boolean and Boolean using `equals` unsafely bypasses 
cooperative equality; use `==` instead
    Error: [ERROR] one error found
    ```
    
https://github.com/apache/celeborn/actions/runs/27126367363/job/80055898287?pr=3720
    
    ### Why are the changes needed?
    
    After this change https://github.com/apache/celeborn/pull/3675, some of 
tests are failing.
    
    ### Does this PR resolve a correctness bug?
    
    - [ ] Yes
    
    ### Does this PR introduce _any_ user-facing change?
    
    - [ ] Yes
    
    ### How was this patch tested?
    
    Existing UTs.
    
    Closes #3722 from s0nskar/CELEBORN-1577_fix_test.
    
    Authored-by: Sanskar Modi <sanskarmod...@gmail.com>
    Signed-off-by: Nicholas Jiang <programg...@163.com>
---
 .../test/scala/org/apache/celeborn/common/util/PbSerDeUtilsTest.scala   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/common/src/test/scala/org/apache/celeborn/common/util/PbSerDeUtilsTest.scala 
b/common/src/test/scala/org/apache/celeborn/common/util/PbSerDeUtilsTest.scala
index 47b618b56..15c4df0fc 100644
--- 
a/common/src/test/scala/org/apache/celeborn/common/util/PbSerDeUtilsTest.scala
+++ 
b/common/src/test/scala/org/apache/celeborn/common/util/PbSerDeUtilsTest.scala
@@ -848,7 +848,7 @@ class PbSerDeUtilsTest extends CelebornFunSuite {
       new TransportMessage(MessageType.HEARTBEAT_FROM_APPLICATION_RESPONSE, 
payload))
       .asInstanceOf[HeartbeatFromApplicationResponse]
     assert(
-      
fromTransportHeartbeatFromApplicationResponse.checkQuotaResponse.isAvailable.equals(true))
+      
fromTransportHeartbeatFromApplicationResponse.checkQuotaResponse.isAvailable)
     
assert(fromTransportHeartbeatFromApplicationResponse.checkQuotaResponse.reason.equals(""))
   }