pareshddevalia commented on code in PR #773:
URL: https://github.com/apache/atlas/pull/773#discussion_r4115778920


##########
dev-support/atlas-docker/Dockerfile.atlas:
##########
@@ -26,18 +26,22 @@ RUN update-java-alternatives --set 
/usr/lib/jvm/java-1.${ATLAS_SERVER_JAVA_VERSI
 
 COPY ./scripts/atlas.sh                                 ${ATLAS_SCRIPTS}/
 COPY ./dist/apache-atlas-${ATLAS_VERSION}-server.tar.gz /home/atlas/dist/
+COPY ./dist/postgresql-42.7.7.jar                       /home/atlas/dist/

Review Comment:
   same as here, why this jar is needed. how it was tested before if postgresql 
as backend were using



##########
dev-support/atlas-docker/scripts/atlas.sh:
##########
@@ -41,6 +41,12 @@ then
   exit 1
 fi
 
+if [ "${ATLAS_BACKEND}" = "postgres" ] && [ -f 
/home/atlas/dist/postgresql-42.7.7.jar ]

Review Comment:
   Why this jar is needed



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