Copilot commented on code in PR #140: URL: https://github.com/apache/gravitino-playground/pull/140#discussion_r2023011203
########## docker-compose.yaml: ########## @@ -165,6 +166,7 @@ services: - "18888:8888" volumes: - ./init/jupyter:/tmp/gravitino + - ./data/jupyter/data:/home/jovyan Review Comment: The mount path for Jupyter includes an extra '/data' subfolder, whereas the PR description specifies using './data/jupyter'. Consider adjusting the volume mapping to './data/jupyter:/home/jovyan' for consistency. ```suggestion - ./data/jupyter:/home/jovyan ``` ########## docker-compose.yaml: ########## @@ -70,6 +70,7 @@ services: volumes: - ./healthcheck:/tmp/healthcheck - ./init/gravitino:/tmp/gravitino + - ./data/gravitino/data:/root/gravitino/data Review Comment: The mount path for Gravitino includes an extra '/data' directory, which does not align with the PR description referring to './data/gravitino'. Consider updating the volume mapping to './data/gravitino:/root/gravitino/data' to ensure consistency. ```suggestion - ./data/gravitino:/root/gravitino/data ``` -- 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: commits-unsubscr...@gravitino.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org