Repository: Alluxio/alluxio (language: java)

Issue:
Update configuration command line tools break to update boolean
**Alluxio Version:**
Master branch & 2.8.2

**Describe the bug**

```
java.lang.IllegalArgumentException: Invalid value for property key XXX: true
	at com.google.common.base.Preconditions.checkArgument(Preconditions.java:435)
	at alluxio.conf.InstancedConfiguration.set(InstancedConfiguration.java:246)
	at alluxio.conf.ServerConfiguration.set(ServerConfiguration.java:112)
	at alluxio.master.meta.DefaultMetaMaster.updateConfiguration(DefaultMetaMaster.java:618)
	at alluxio.master.meta.MetaMasterConfigurationServiceHandler.lambda$updateConfiguration$5(MetaMasterConfigurationServiceHandler.java:139)
	at alluxio.RpcUtils.callAndReturn(RpcUtils.java:122)
	at alluxio.RpcUtils.call(RpcUtils.java:83)
	at alluxio.RpcUtils.call(RpcUtils.java:58)
	at alluxio.master.meta.MetaMasterConfigurationServiceHandler.updateConfiguration(MetaMasterConfigurationServiceHandler.java:137)
	at alluxio.grpc.MetaMasterConfigurationServiceGrpc$MethodHandlers.invoke(MetaMasterConfigurationServiceGrpc.java:585)
	at io.grpc.stub.ServerCalls$UnaryServerCallHandler$UnaryServerCallListener.onHalfClose(ServerCalls.java:182)
	at io.grpc.internal.ServerCallImpl$ServerStreamListenerImpl.halfClosed(ServerCallImpl.java:331)
	at io.grpc.internal.ServerImpl$JumpToApplicationThreadServerStreamListener$1HalfClosed.runInContext(ServerImpl.java:797)
	at io.grpc.internal.ContextRunnable.run(ContextRunnable.java:37)
	at io.grpc.internal.SerializingExecutor.run(SerializingExecutor.java:123)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:834)
```

**To Reproduce**

```bash
bin/alluxio fsadmin updateConf xxx=true
```

**Expected behavior**

Update conf successfully

**Urgency**

Urgent.

**Are you planning to fix it**

Yeah

**Additional context**
No



Discussion / hints:

Fix patch:
diff --git a/core/server/master/src/main/java/alluxio/master/meta/DefaultMetaMaster.java b/core/server/master/src/main/java/alluxio/master/meta/DefaultMetaMaster.java
index ca0b1c434d64..bba47ec52bea 100644
--- a/core/server/master/src/main/java/alluxio/master/meta/DefaultMetaMaster.java
+++ b/core/server/master/src/main/java/alluxio/master/meta/DefaultMetaMaster.java
@@ -629,7 +629,8 @@ public Map<String, Boolean> updateConfiguration(Map<String, String> propertiesMa
         if (Configuration.getBoolean(PropertyKey.CONF_DYNAMIC_UPDATE_ENABLED)
             && key.isDynamic()) {
           Object oldValue = Configuration.get(key);
-          Configuration.set(key, entry.getValue(), Source.RUNTIME);
+          Object value = key.parseValue(entry.getValue());
+          Configuration.set(key, value, Source.RUNTIME);
           result.put(entry.getKey(), true);
           successCount++;
           LOG.info("Property {} has been updated to \"{}\" from \"{}\"",


Test patch: