AnzhiZhang opened a new issue, #3816:
URL: https://github.com/apache/texera/issues/3816

   ### Proposed Changes
   
   It might be good to improve the Docker Compose file for LakeFS in our 
`file-service`. To be specific:
   
   1. Remove `version`. In the current version of Docker Compose, the version 
is obsolete. We are not including this in the single node installation compose 
file. See https://docs.docker.com/reference/compose-file/version-and-name/
   2. Update `container_name`. Currently, we have `minio`, `postgres`, and 
`lakefs` as container names. However, this name is globally on a machine. We 
should give a namespace to avoid collisions and inconvenience. Similarly to 
what we did in the single node configuration, I propose `texera-lakefs-minio` 
and the others.
   3. Give default `volumes`. Currently, minio and postgres data are persisted 
using auto-created Docker volumes, which will be a generated hash name. We have 
a comment in the Docker Compose file to help developers assign a physical 
location to persist them. We can set default volumes like `minio_data` to 
better persist data and prevent loss.
   
   <img width="2872" height="210" alt="Image" 
src="https://github.com/user-attachments/assets/81cbe4b2-ddd5-4aef-950d-56ee239f86c6";
 />
   
   <img width="1486" height="350" alt="Image" 
src="https://github.com/user-attachments/assets/43ed7e9d-f0a5-412f-84e0-58d5c8624824";
 />
   
   ### Changes for Developers to Apply
   
   After this change, developers need to:
   
   - Rename their auto-generated volumes to the new name.
   - Recreate containers.


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

Reply via email to