[GitHub] [incubator-pinot] adriancole opened a new issue #5975: Allow relative paths for properties that configure directories

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

adriancole opened a new issue #5975:
URL: https://github.com/apache/incubator-pinot/issues/5975


   relative paths allow us to build a fully configured layer more easily as it can be contained under a directory besides /. For example, we can add data under a /pinot/data/service directory which can make sense in docker when there's only 1 pid running anyway. This is particularly helpful in ServiceManager as it allows us to prepare a fully configured image including schema as a layer.
   
   Right now it seems at least these need to be URLS or absolute paths
   
   controller.data.dir pinot.server.instance.dataDir and pinot.server.instance.segmentTarDir
   
   @kbastani mentioned there are test that cover currently allowed things here
   https://github.com/apache/incubator-pinot/pull/5639/commits/a6366a0087fd68c943d67b8fde2eba01e27b3b75#diff-72e3ceeda2eb1099c4746aedd0e3cfdcR309
   
   Specifically, I would like to specify similar to Kafka or ZooKeeper something like
   
   `pinot.server.instance.dataDir=./data/server`
   
   I don't mind if the syntax has a leading `./` or not.
   
   


----------------------------------------------------------------
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.

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org