[
https://issues.apache.org/jira/browse/PHOENIX-6633?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17771085#comment-17771085
]
ASF GitHub Bot commented on PHOENIX-6633:
-----------------------------------------
stoty commented on PR #108:
URL:
https://github.com/apache/phoenix-connectors/pull/108#issuecomment-1742991857
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Comment |
|:----:|----------:|--------:|:--------|
| +0 :ok: | reexec | 0m 13s | Docker mode activated. |
||| _ Prechecks _ |
| +1 :green_heart: | dupname | 0m 0s | No case conflicting files
found. |
| +1 :green_heart: | hbaseanti | 0m 0s | Patch does not have any
anti-patterns. |
| +1 :green_heart: | @author | 0m 0s | The patch does not contain any
@author tags. |
| +1 :green_heart: | test4tests | 0m 0s | The patch appears to include
1 new or modified test files. |
||| _ master Compile Tests _ |
| +0 :ok: | mvndep | 11m 4s | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 8m 40s | master passed |
| +1 :green_heart: | compile | 1m 17s | master passed |
| +1 :green_heart: | checkstyle | 0m 15s | master passed |
| +1 :green_heart: | javadoc | 0m 58s | master passed |
| +0 :ok: | spotbugs | 0m 53s | phoenix5-spark in master has 40 extant
spotbugs warnings. |
| +0 :ok: | spotbugs | 0m 41s | phoenix5-spark3 in master has 22 extant
spotbugs warnings. |
| +0 :ok: | spotbugs | 1m 39s | root in master has 111 extant spotbugs
warnings. |
||| _ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 11s | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 7m 48s | the patch passed |
| +1 :green_heart: | compile | 1m 27s | the patch passed |
| -1 :x: | javac | 1m 27s | root generated 118 new + 1999 unchanged -
125 fixed = 2117 total (was 2124) |
| -1 :x: | checkstyle | 0m 15s | root: The patch generated 2 new + 158
unchanged - 9 fixed = 160 total (was 167) |
| -1 :x: | markdownlint | 0m 4s | The patch generated 10 new + 91
unchanged - 14 fixed = 101 total (was 105) |
| -1 :x: | whitespace | 0m 0s | The patch has 2 line(s) that end in
whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer
https://git-scm.com/docs/git-apply |
| +1 :green_heart: | xml | 0m 2s | The patch has no ill-formed XML
file. |
| -1 :x: | javadoc | 0m 12s | phoenix5-spark generated 1 new + 151
unchanged - 5 fixed = 152 total (was 156) |
| -1 :x: | javadoc | 0m 11s | phoenix5-spark3 generated 1 new + 187
unchanged - 4 fixed = 188 total (was 191) |
| -1 :x: | javadoc | 0m 27s | root generated 2 new + 2128 unchanged - 9
fixed = 2130 total (was 2137) |
| +1 :green_heart: | spotbugs | 3m 35s | the patch passed |
||| _ Other Tests _ |
| -1 :x: | unit | 55m 28s | root in the patch failed. |
| -1 :x: | asflicense | 0m 26s | The patch generated 3 ASF License
warnings. |
| | | 97m 1s | |
| 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-108/4/artifact/yetus-general-check/output/Dockerfile
|
| GITHUB PR | https://github.com/apache/phoenix-connectors/pull/108 |
| Optional Tests | dupname asflicense javac javadoc unit spotbugs hbaseanti
checkstyle compile markdownlint xml |
| uname | Linux 6703f5426617 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19
13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | dev/phoenix-connectors-personality.sh |
| git revision | master / ca76ff8 |
| Default Java | Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08 |
| javac |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-108/4/artifact/yetus-general-check/output/diff-compile-javac-root.txt
|
| checkstyle |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-108/4/artifact/yetus-general-check/output/diff-checkstyle-root.txt
|
| markdownlint |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-108/4/artifact/yetus-general-check/output/diff-patch-markdownlint.txt
|
| whitespace |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-108/4/artifact/yetus-general-check/output/whitespace-eol.txt
|
| javadoc |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-108/4/artifact/yetus-general-check/output/diff-javadoc-javadoc-phoenix5-spark.txt
|
| javadoc |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-108/4/artifact/yetus-general-check/output/diff-javadoc-javadoc-phoenix5-spark3.txt
|
| javadoc |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-108/4/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-108/4/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-108/4/testReport/
|
| asflicense |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-108/4/artifact/yetus-general-check/output/patch-asflicense-problems.txt
|
| Max. process+thread count | 1958 (vs. ulimit of 30000) |
| modules | C: phoenix5-spark phoenix5-spark3 phoenix5-spark3-it . U: . |
| Console output |
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-108/4/console
|
| versions | git=2.7.4 maven=3.3.9 spotbugs=4.1.3 markdownlint=0.22.0 |
| Powered by | Apache Yetus 0.12.0 https://yetus.apache.org |
This message was automatically generated.
> Use standard JDBC URL in Spark Connector and make it optional
> -------------------------------------------------------------
>
> Key: PHOENIX-6633
> URL: https://issues.apache.org/jira/browse/PHOENIX-6633
> Project: Phoenix
> Issue Type: New Feature
> Reporter: Istvan Toth
> Assignee: Istvan Toth
> Priority: Major
>
> We require explicitly specifying the ZK URL for the Phoenix-Spark connector.
> Since having hbase-site.xml on the classpath is required by Phoenix anyway,
> we could use the quorum from hbase-site.xml as default, which would make life
> easier for users.
> Phoenix already handles this, we just need to remove the check, and adopt the
> JDBC URL not to specify the ZK quorum if missing.
> The current zkUrl code also acts as an aritificial limitation on using the
> new zookeeperless Hbase registries from Spark.
> We need to deprecate zkUrl, and add a new jdbcUrl property, which is simply
> passed to Phoenix without further processing.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
