Re: [PR] Fix for NPE in subqueries in SSE when OOM protection is ON [pinot]

[via GitHub]

vrajat commented on PR #15046:
URL: https://github.com/apache/pinot/pull/15046#issuecomment-2655415009

   @Jackie-Jiang, +1 to what @yashmayya said. The "refactor" is the fix. It 
mostly consists of whitespace changes. The body of the function _has to be_ 
moved out of the try/finally block. That will change whitespace indentation.  
Check the screenshot of the diff when whitespace is removed.
   
   ![Screenshot 2025-02-13 at 9 20 56 
AM](https://github.com/user-attachments/assets/f78da5b4-a49a-492c-b881-2595b9dcc7c9)
   
   


-- 
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: commits-unsubscr...@pinot.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org