Re: [PR] [#6237]fix: add missing @override annotations [gravitino]

2025-01-15 Thread via GitHub


jerryshao merged PR #6257:
URL: https://github.com/apache/gravitino/pull/6257


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



Re: [PR] [#6237]fix: add missing @override annotations [gravitino]

2025-01-15 Thread via GitHub


jerryshao closed pull request #6257: [#6237]fix: add missing @override 
annotations
URL: https://github.com/apache/gravitino/pull/6257


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



[PR] [#6237]fix: add missing @override annotations [gravitino]

2025-01-15 Thread via GitHub


github-actions[bot] opened a new pull request, #6257:
URL: https://github.com/apache/gravitino/pull/6257

   
   
   ### What changes were proposed in this pull request?
   
   Add missing `@override` annotations
   
   ### Why are the changes needed?
   
   Fix:  https://github.com/apache/gravitino/issues/6237
   
   ### Does this PR introduce _any_ user-facing change?
   N/A
   
   ### How was this patch tested?
   N/A


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



Re: [PR] [#6237]fix: add missing @override annotations [gravitino]

2025-01-15 Thread via GitHub


jerryshao merged PR #6244:
URL: https://github.com/apache/gravitino/pull/6244


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



Re: [PR] [#6237]fix: add missing @override annotations [gravitino]

2025-01-14 Thread via GitHub


sunxiaojian commented on PR #6244:
URL: https://github.com/apache/gravitino/pull/6244#issuecomment-2591816759

   @justinmclean PTAL , thanks


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



[PR] [#6237]fix: add missing @override annotations [gravitino]

2025-01-14 Thread via GitHub


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

   
   
   ### What changes were proposed in this pull request?
   
   Add missing @override annotations
   
   ### Why are the changes needed?
   
   (Please clarify why the changes are needed. For instance,
 1. If you propose a new API, clarify the use case for a new API.
 2. If you fix a bug, describe the bug.)
   
   Fix:  https://github.com/apache/gravitino/issues/6237
   
   ### Does this PR introduce _any_ user-facing change?
   N/A
   
   ### How was this patch tested?
   N/A


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