attilapiros commented on pull request #28708:
URL: https://github.com/apache/spark/pull/28708#issuecomment-638916984


   I just found a bug in my `BlockManagerSuite` test which is caused by this 
line: 
   
   
https://github.com/apache/spark/blob/ccb8827c61e3178c83ae505dbddd30b1a6a2d361/core/src/main/scala/org/apache/spark/shuffle/IndexShuffleBlockResolver.scala#L54
   
   During offloading once this should refer to the source block manager and 
once the target.
   I do not know how it was running yesterday (today I rebased on top of the 
new changes) successfully but it was green. 
   
   Anyway I would like to answer your last question but for this I needed to do 
some more testing.
    
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to