dongjoon-hyun commented on pull request #35190:
URL: https://github.com/apache/spark/pull/35190#issuecomment-1012707127


   In your branch, the ownership was clean line by line. Let me check the bug 
in our merge script.
   ```
   7fd361973d2 (Liang-Chi Hsieh 2021-12-23 19:41:02 -0800 272)     override def 
append(loggingEvent: LogEvent): Unit = loggingEvent.synchronized {
   7fd361973d2 (Liang-Chi Hsieh 2021-12-23 19:41:02 -0800 273)       val 
copyEvent = loggingEvent.toImmutable
   7fd361973d2 (Liang-Chi Hsieh 2021-12-23 19:41:02 -0800 274)       if 
(copyEvent.getLevel.isMoreSpecificThan(_threshold)) {
   0e3dcdbd7f1 (yangjie01       2022-01-13 17:22:53 +0800 275)         
_loggingEvents.synchronized {
   0e3dcdbd7f1 (yangjie01       2022-01-13 17:22:53 +0800 276)           if 
(_loggingEvents.size >= maxEvents) {
   0e3dcdbd7f1 (yangjie01       2022-01-13 17:22:53 +0800 277)             val 
loggingInfo = if (msg == "") "." else s" while logging $msg."
   0e3dcdbd7f1 (yangjie01       2022-01-13 17:22:53 +0800 278)             
throw new IllegalStateException(
   0e3dcdbd7f1 (yangjie01       2022-01-13 17:22:53 +0800 279)               
s"Number of events reached the limit of $maxEvents$loggingInfo")
   0e3dcdbd7f1 (yangjie01       2022-01-13 17:22:53 +0800 280)           }
   0e3dcdbd7f1 (yangjie01       2022-01-13 17:22:53 +0800 281)           
_loggingEvents.append(copyEvent)
   ca5cabe5fbf (Liang-Chi Hsieh 2021-12-17 21:40:51 -0800 282)         }
   88fc8dbc09c (Maxim Gekk      2020-01-14 16:03:10 +0900 283)       }
   88fc8dbc09c (Maxim Gekk      2020-01-14 16:03:10 +0900 284)     }
   ```


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

Reply via email to