Correct the way an empty result is interpreted (#1828)

FoundationDB · Oct 9, 2023 · 05c98d7 · 05c98d7
1 parent 59b8d2c
commit 05c98d7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/fdbclient/admin_client.go b/fdbclient/admin_client.go
@@ -352,6 +352,7 @@ func (client *cliAdminClient) GetMaintenanceZone() (string, error) {
 	if err != nil {
 		return "", err
 	}
+
 	return string(mode), nil
 }
 

diff --git a/fdbclient/fdb_client.go b/fdbclient/fdb_client.go
@@ -63,8 +63,9 @@ func (fdbClient *realFdbLibClient) getValueFromDBUsingKey(fdbKey string, timeout
 		}
 
 		rawResult := transaction.Get(fdb.Key(fdbKey)).MustGet()
+		// If the value is empty return an empty byte slice. Otherwise, an error will be thrown.
 		if len(rawResult) == 0 {
-			return nil, err
+			return []byte{}, err
 		}
 
 		return rawResult, err
-Original file line number
+Diff line change
@@ Expand Up @@
     	if err != nil {
     		return "", err
     	}
     	return string(mode), nil
     }
@@ Expand Down @@