[
https://issues.apache.org/jira/browse/PHOENIX-6923?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17788443#comment-17788443
]
ASF GitHub Bot commented on PHOENIX-6923:
-----------------------------------------
stoty commented on PR #114:
URL:
https://github.com/apache/phoenix-connectors/pull/114#issuecomment-1821086734
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 3m 25s | Docker mode activated. |
||| _ Prechecks _ |
| +1 :green_heart: | dupname | 0m 0s | No case conflicting files
found. |
| +1 :green_heart: | @author | 0m 0s | The patch does not contain any
@author tags. |
| -1 :x: | test4tests | 0m 0s | The patch doesn't appear to include
any new or modified tests. Please justify why no new tests are needed for this
patch. Also please list what manual steps were performed to verify this patch.
|
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 10m 22s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 8m 24s | master passed |
| +1 :green_heart: | compile | 1m 10s | master passed |
| +1 :green_heart: | javadoc | 0m 40s | master passed |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 10s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 17m 15s | the patch passed |
| +1 :green_heart: | compile | 1m 16s | the patch passed |
| +1 :green_heart: | javac | 1m 16s | the patch passed |
| +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace
issues. |
| +1 :green_heart: | xml | 0m 3s | The patch has no ill-formed XML
file. |
| -1 :x: | javadoc | 0m 21s | root generated 20 new + 1401 unchanged -
20 fixed = 1421 total (was 1421) |
||| _ Other Tests _ |
| -1 :x: | unit | 18m 48s | root in the patch failed. |
| -1 :x: | asflicense | 0m 18s | The patch generated 3 ASF License
warnings. |
| | | 62m 57s | |
| Subsystem | Report/Notes |
|----------:|:-------------|
| Docker | ClientAPI=1.43 ServerAPI=1.43 base:
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-114/1/artifact/yetus-general-check/output/Dockerfile
|
| GITHUB PR | https://github.com/apache/phoenix-connectors/pull/114 |
| Optional Tests | dupname asflicense javac javadoc unit xml compile |
| uname | Linux 7a3b76b73c33 5.15.0-88-generic #98-Ubuntu SMP Mon Oct 2
15:18:56 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev/phoenix-connectors-personality.sh |
| git revision | master / 26c1aba |
| Default Java | Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08 |
| javadoc |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-114/1/artifact/yetus-general-check/output/diff-javadoc-javadoc-root.txt
|
| unit |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-114/1/artifact/yetus-general-check/output/patch-unit-root.txt
|
| Test Results |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-114/1/testReport/
|
| asflicense |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-114/1/artifact/yetus-general-check/output/patch-asflicense-problems.txt
|
| Max. process+thread count | 1806 (vs. ulimit of 30000) |
| modules | C: phoenix5-spark phoenix5-spark3-it . U: . |
| Console output |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-114/1/console
|
| versions | git=2.7.4 maven=3.3.9 |
| Powered by | Apache Yetus 0.12.0 https://yetus.apache.org |
This message was automatically generated.
> Always run Scala tests for Phoenix-Spark connector
> --------------------------------------------------
>
> Key: PHOENIX-6923
> URL: https://issues.apache.org/jira/browse/PHOENIX-6923
> Project: Phoenix
> Issue Type: Bug
> Components: connectors, spark-connector
> Affects Versions: connectors-6.0.0
> Reporter: Istvan Toth
> Assignee: Istvan Toth
> Priority: Major
>
> When Richard fixed the Scala tests in PHOENIX-6361, these were gated behind a
> profile, as these tests really mess up the maven output.
> However, those tests are too important to skip, so we should run them by
> default anyway.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
