PakhomovAlexander commented on code in PR #1274:
URL: https://github.com/apache/ignite-3/pull/1274#discussion_r1032174994
##########
packaging/zip/ignite3db:
##########
@@ -33,12 +32,21 @@ if [ -z ${IGNITE_HOME+x} ]; then IGNITE_HOME=$(pwd); fi
cd ${IGNITE_HOME} || exit
start() {
+ echo "Starting Ignite 3..."
+
CMD="${JAVA_CMD_WITH_ARGS} ${APPLICATION_ARGS}"
$CMD >>/dev/null 2>&1 </dev/null & echo $! >${IGNITE_HOME}/pid
+
+ rest_address_file=${WORK_DIR}/rest-address
+ while [ ! -f "$rest_address_file" ]; do sleep 0.5; done
+ rest_address=$(cat "$rest_address_file")
+
+ echo "Node named ${NODE_NAME} started successfully. REST address is
http://$rest_address";
Review Comment:
During implementation, I've found that there is no clear way to get the JDBC
connection string until the node is joined to the logical topology. But we
cannot wait for that. To form a logical topology a user should know the REST
address. Well, I've decided to design new command to print JDBC connection
string but not I would suggest leaving the PR as it is.
--
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]
