(phoenix) branch master updated: PHOENIX-7194 Yetus does not run tests in phoenix-core if phoenix-core-client or phoenix-core-server changes

[stoty]

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}