ramanathan1504 commented on issue #4262:
URL: 
https://github.com/apache/logging-log4j2/issues/4262#issuecomment-5572953959

   @abysas @abhinav-phi  Thanks!🙏
   
   Confirmed 2.24.1 → 3.0.0-SNAPSHOT. It is three attributes, not one: 
`bufferedIo` and `bufferSize`                                                   
                          
     are dropped by `build()` the same way, and nothing warns.                  
                                                                                
                   
                                                                                
                                                                                
                   
     Fix:                                                                       
                                                                                
                   
     1. Pass `isImmediateFlush()` into the constructor instead of `true`.       
                                                                                
                   
     2. Keep `createDefaultAppenderForLayout` on `true`.                        
                                                                                
                   
     3. `bufferSize` is a decision, not a patch — the manager key is 
`target.follow.direct`, so two                                                  
                              
        Console appenders share a manager and the second value is lost anyway. 
Widen the key or drop                                                           
                    
        the docs row.                                                           
                                                                                
                   
     4. Same change on `main`; one-line changelog entry.                        
                                                                                
                   
                                                                                
                                                                                
                   
     **Red-green:** assert `immediateFlush` is `false` when configured false.   
                                                                                
                   
     **Coverage:** `ConsoleAppenderTest` — configured false, omitted default 
true.                                                                           
                      
                                                                                
                                                                                
                   
     Default stays `true`, so it is compatible — but `System.out` is an 
autoflushing `PrintStream`, so                                                  
                           
     honouring `false` genuinely withholds output.


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

Reply via email to