a638011 opened a new pull request, #10130:
URL: https://github.com/apache/gravitino/pull/10130

   ## What changes were proposed in this pull request?
   
   Add validation for GRAVITINO_HOME environment variable before building 
relative storage paths in H2Database.
   
   ## Why are the changes needed?
   
   Currently, if GRAVITINO_HOME is not set and a relative storage path is 
configured, the code throws a NullPointerException at runtime during H2 
initialization. This makes debugging difficult and causes startup failures.
   
   Fix #10128
   
   ## Does this PR introduce any user-facing change?
   
   Yes - users will now get a clear IllegalArgumentException with a descriptive 
message when GRAVITINO_HOME is missing, instead of a confusing 
NullPointerException.
   
   ## How was this patch tested?
   
   - Added unit test 
TestH2Database.testGetStoragePathWithRelativePathWhenGravitinoHomeMissing()
   - Test verifies that IllegalArgumentException is thrown when GRAVITINO_HOME 
is unset with relative path
   - Existing tests should continue to pass


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