saptarshi1996 opened a new pull request, #15680:
URL: https://github.com/apache/pinot/pull/15680

   ### Prevent Runtime Errors by Validating Schema Fields
   
   This pull request addresses a potential runtime error in the isObjEmpty 
function by adding a validation check to ensure that each field in the 
schemaConfig object exists before accessing its length property.
   
   **Changes Made**
   
   <img width="450" alt="Screenshot 2025-04-30 at 20 40 22" 
src="https://github.com/user-attachments/assets/d90a39ba-9874-4034-bdb6-ee16222acc8d";
 />
   
   
   **Reason for Change**
   
   Previously, if any of the specified fields (dimensionFieldSpecs, 
metricFieldSpecs, or dateTimeFieldSpecs) were undefined or null, accessing 
their length property would result in a runtime error. This change ensures that 
the field exists and is an array before attempting to access its length, 
thereby enhancing the robustness of the function.
   
   **Impact**
   
   This is a non-breaking change that improves error handling in the isObjEmpty 
function. It does not introduce any new features or alter existing 
functionality beyond the added validation.


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

Reply via email to