jerqi commented on PR #1121:
URL:
https://github.com/apache/incubator-uniffle/pull/1121#issuecomment-1674148003
> Hi, all. Current version solve the problem that remote storage config
contain ',', but cant solve the problem container '=' or ';'.
>
> For now, Three ways we were summarized:
>
> ### (1) add escape symbol before ',' and '='
> We cant add '\' or '`' before ',' and '='. This way is not graceful. I
think this way should be obsolete.
>
> ### (2) add new rss.coordinator.remote.storage.conf.{cluster}
> Add new config, format is ` rss.coordinator.remote.storage.conf.{cluster}
key1=value1,value11;key2=value2` The format is different from
`rss.coordinator.remote.storage.cluster.conf` But cant solve the problem
container '=' or ';'.
>
> ### (3) add new config file
> Add new config, format is
`rss.coordinator.remote.storage.conf.path.{cluster} file://xxxx` The remote
storage configs are stored in file://xxxx.
>
> @jerqi @zuston @LuciferYang @advancedxy Which proposal do you prefer? Or a
better proposal?
I prefer option 2.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
