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

   ### What changes were proposed in this pull request?
   
   This PR renames the `userRoleRelList` parameter to `userRoleRels` in the 
`batchInsertUserRoleRel` method of `UserRoleRelBaseSQLProvider.java`. This 
change applies to both the Java method parameter and its `@Param` annotation to 
match the collection name used within the MyBatis SQL template.
   
   ### Why are the changes needed?
   
   The mismatch between the parameter name defined in the Java code 
(`userRoleRelList`) and the name expected by the SQL template (`userRoleRels`) 
would cause a runtime error when the batch insert operation is called [cite].
   
   This fix resolves the bug and also makes the method signature consistent 
with other methods in the same file, such as 
`batchInsertUserRoleRelOnDuplicateKeyUpdate` [cite].
   
   Fix: #8659
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. This is an internal fix to the storage layer and does not alter any 
user-facing APIs or behavior.
   
   ### How was this patch tested?
   
   The fix was verified by running the complete test suite for the `:core` 
module. All existing tests 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