This is an automated email from the ASF dual-hosted git repository.
stoty pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/phoenix.git
The following commit(s) were added to refs/heads/master by this push:
new 802d30938a PHOENIX-7194 Yetus does not run tests in phoenix-core if
phoenix-core-client or phoenix-core-server changes
802d30938a is described below
commit 802d30938acc175c4da1b74cb1946d22bcb60de8
Author: Istvan Toth <st...@apache.org>
AuthorDate: Thu Feb 1 07:45:12 2024 +0100
PHOENIX-7194 Yetus does not run tests in phoenix-core if
phoenix-core-client or phoenix-core-server changes
---
dev/phoenix-personality.sh | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/dev/phoenix-personality.sh b/dev/phoenix-personality.sh
index 7c3b360de6..706cfb85b3 100755
--- a/dev/phoenix-personality.sh
+++ b/dev/phoenix-personality.sh
@@ -155,9 +155,6 @@ function personality_modules
# I have been unable to get Jacoco running reliably on ASF Jenkins, thus it
is disabled.
extra="--threads=1 -DPhoenixPatchProcess -Dskip.code-coverage "
- if [[ "${PATCH_BRANCH}" = 4* ]]; then
- extra="${extra} -Dhttps.protocols=TLSv1.2"
- fi
# If we have HBASE_PROFILE specified pass along
# the hbase.profile system property.
@@ -182,6 +179,12 @@ function personality_modules
MODULES=(.)
fi
+ # Yetus logic cannot handle the tests for a module being in another module,
which has been the
+ # case since we split the client and server code, so always run the UTs/ITs
on the root.
+ if [[ ${testtype} == unit ]]; then
+ MODULES=(.)
+ fi
+
if [[ ${testtype} == mvninstall ]]; then
# shellcheck disable=SC2086
personality_enqueue_module . ${extra}
