[GitHub] [incubator-uniffle] qijiale76 commented on issue #1051: [Improvement] [mr] Merge mrJobConf and rssJobConf at RssShuffle.java

["qijiale76 (via GitHub)" <gi...@apache.org>]

qijiale76 commented on issue #1051:
URL: https://github.com/apache/incubator-uniffle/issues/1051#issuecomment-1655322233

   Now `mapreduce.rss.client.type` and `mapreduce.rss.remote.storage.path` must be configued at `mapred-site.xml` instead of `dynamic_client.conf`. It should also be allowd that these conf are not configued at `mapred-site.xml`.
   ```
   Exception in thread "main" java.lang.IllegalArgumentException: The value of null should be one of [GRPC_NETTY, GRPC]
   ```
   ```
   Exception in thread "main" java.lang.IllegalStateException: Can't find remoteStorage: with storageType[MEMORY_LOCALFILE_HDFS]
   ```


-- 
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: dev-unsubscribe@uniffle.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org