[GitHub] [incubator-kvrocks] tisonkun commented on pull request #850: kvrocks docker can use environment variables define config

[GitBox <gi...@apache.org>]

tisonkun commented on PR #850:
URL: https://github.com/apache/incubator-kvrocks/pull/850#issuecomment-1242899313

   @tanruixiang I suggest we focus on the design decision before jump into implementation details.
   
   @PragmaTwice @git-hulk @ShooterIT @usrtax a quick idea on passing override from environment variable is taking a namespace said `kvrocks.` and when start the server, read from the env vars and overwrite `xxx` with `kvrocks.xxx`.
   
   Anyway, I hope this initiative goes through a design instead of rushing into patches.


-- 
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: issues-unsubscribe@kvrocks.apache.org

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