[GitHub] [incubator-uniffle] zhengchenyu commented on pull request #1131: [#1115] improvement: Unregister shuffle explicitly when application i…

["zhengchenyu (via GitHub)" <gi...@apache.org>]

zhengchenyu commented on PR #1131:
URL: https://github.com/apache/incubator-uniffle/pull/1131#issuecomment-1674269787

   > Could we add UT for this pr?
   
   I am confused about UT. This PR make effect in main. For now, I did not find a good way to construct UT yet.
   I think if integration test pass, there will be no any problem.


-- 
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: issues-unsubscribe@uniffle.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@uniffle.apache.org
For additional commands, e-mail: issues-help@uniffle.apache.org