diqiu50 commented on code in PR #4106:
URL: https://github.com/apache/gravitino/pull/4106#discussion_r1683948311
##########
integration-test-common/src/test/java/org/apache/gravitino/integration/test/container/HiveContainer.java:
##########
@@ -163,15 +163,21 @@ protected boolean checkContainerStatus(int retryLimit) {
"CREATE TABLE IF NOT EXISTS default.employee ( eid int, name String, "
+ "salary String, destination String) ";
await()
- .atMost(30, TimeUnit.SECONDS)
- .pollInterval(30 / retryLimit, TimeUnit.SECONDS)
+ .atMost(150, TimeUnit.SECONDS)
+ .pollInterval(150 / retryLimit, TimeUnit.SECONDS)
.until(
Review Comment:
The check frequency may be too slow
##########
dev/docker/kerberos-hive/start.sh:
##########
@@ -75,26 +75,55 @@ sed -i "s/mockhost/${HOSTNAME}/g"
${HIVE_HOME}/conf/hive-site.xml
# format HDFS
${HADOOP_HOME}/bin/hdfs namenode -format -nonInteractive
+# Wait the format operation exit total.
+sleep 2
+# Check Whether the ports are occupied and kill the process
+for port in 3306 9000 9083 10000 10002 50070 50075 50010
+do
+ echo "Check whether the port ${port} is occupied..."
+ pid=$(netstat -tlnp | grep ${port} | awk '{print $7}' | awk -F '/' '{print
$1}')
+ # if pid is not null, kill the process
+ if [[ -n ${pid} ]]; then
+ ps -ef | grep ${pid}
+ echo "The port ${port} is occupied, the process id is ${pid}, killing the
process..."
+ kill -9 ${pid}
+ fi
+done
+
echo "Starting HDFS..."
Review Comment:
I think the port conflict maybe cause by
https://www.ncftp.com/ncftpd/doc/misc/ephemeral_ports.html.
Killing the process may not solve the problem, or it may cause other
problems.
--
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]
