This is an automated email from the ASF dual-hosted git repository.

jshao pushed a commit to branch branch-0.8
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/branch-0.8 by this push:
     new 79e8804d04 [Minor] Optionals should not be set to null (#6333)
79e8804d04 is described below

commit 79e8804d04dc777938558d356e7746fc5a94a200
Author: github-actions[bot] 
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Mon Jan 20 11:49:09 2025 +0800

    [Minor] Optionals should not be set to null (#6333)
    
    ### What changes were proposed in this pull request?
    
    Optionals should not be set to null
    
    ### Why are the changes needed?
    
    see above
    
    Fix: # N/A
    
    ### Does this PR introduce _any_ user-facing change?
    
    No
    
    ### How was this patch tested?
    
    Tested locally
    
    Co-authored-by: Justin Mclean <[email protected]>
---
 .../src/main/java/org/apache/gravitino/cli/commands/OwnerDetails.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/clients/cli/src/main/java/org/apache/gravitino/cli/commands/OwnerDetails.java 
b/clients/cli/src/main/java/org/apache/gravitino/cli/commands/OwnerDetails.java
index 8485a58756..a815d6ba14 100644
--- 
a/clients/cli/src/main/java/org/apache/gravitino/cli/commands/OwnerDetails.java
+++ 
b/clients/cli/src/main/java/org/apache/gravitino/cli/commands/OwnerDetails.java
@@ -68,7 +68,7 @@ public class OwnerDetails extends Command {
   /** Displays the owner of an entity. */
   @Override
   public void handle() {
-    Optional<Owner> owner = null;
+    Optional<Owner> owner = Optional.empty();
     MetadataObject metadata = MetadataObjects.parse(entity, entityType);
 
     try {

Reply via email to