dimas-b commented on PR #1589:
URL: https://github.com/apache/polaris/pull/1589#issuecomment-2902242067

   > Consolidate the shared libs between Admin Tool and Server
   
   I'm a bit lost now as to the purpose of the archive-based distribution. 
   
   From my POV, the archive is meant mostly for local installs where the user 
runs both the server and the admin tool manually. In this case, mixing admin 
and server stuff does not look problematic to me.
   
   For automated deployments, I'd expect users to use docker.
   
   If a user needs to build a custom docker image, I believe the process should 
be to start from the source, not from the archive-based distribution. 
   
   On the other hand, if we expect users to use the archive for deployments 
(e.g. on bare metal), then we should definitely keep the admin and server 
separate... I personally would not expect much user demand for this use case, 
though.
   
   All in all, as far as this PR is concerned, I suppose we can have a solution 
that allows both use cases:
   1. Have one distribution archive
   2. Make `admin` and `server` directories usable independently
       * provide separate startup scripts in `admin/bin/polaris-admin` and 
`server/bin/polaris-server`
   3. Do _not_ use `run.sh` at the root of the archive


-- 
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-unsubscr...@polaris.apache.org

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

Reply via email to