pareshddevalia opened a new pull request, #780:
URL: https://github.com/apache/atlas/pull/780
…recognized --add-opens JVM options
Changes:- Remove redundant REST_SERVER_OPTS from compose; document REST
server startup in README.
## What changes were proposed in this pull request?
Removed hardcoded REST_SERVER_OPTS (--add-opens=java.base/...) from
dev-support/atlas-docker/docker-compose.atlas-rest.yml. Those JVM flags are
already applied by distro/src/bin/rest_start.py via get_default_jvm_opts() when
the runtime Java version is 9 or newer. Forcing them in Compose caused JVM
startup failure on Java 8 bases (Unrecognized option: --add-opens=...) before
RestNotificationMain could start.
Added step 5 to dev-support/atlas-docker/README.md describing how to build
and start the Atlas REST notification server (atlas-rest) after the main Atlas
stack is up, including compose variants when Atlas was started with the
Postgres or HBase backend overlays.
## How was this patch tested?
Maven: Ran a full build (mvn clean verify / atlas-build docker flow) to
produce dist/apache-atlas-${ATLAS_VERSION}-rest-server.tar.gz.
Docker – Java 8, 17: Built atlas-base built and started the stack per README
(Atlas backend + atlas-rest). Confirmed the REST notification server starts
without JVM errors, listens on port 41000, and admin/status is reachable (e.g.
curl -u admin:atlasR0cks! http://localhost:41000/rest/api/atlas/admin/status).
--
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]
