Skip to content

PHOENIX-7625 Adding query plan information to connection Activity Logger #2177

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

PHOENIX-7625 Adding query plan information to connection Activity Logger #2177

wants to merge 6 commits into from
+115 −1

Conversation

@vmeka2020
Copy link
Contributor

@vmeka2020 vmeka2020 commented Jun 5, 2025 •
edited
Loading

Open source JIRA : PHOENIX-7625

@vmeka2020 vmeka2020 changed the title Adding query plan information to connection Activity Logger PHOENIX-7625 Adding query plan information to connection Activity Logger Jun 5, 2025
@virajjasani virajjasani self-requested a review June 5, 2025 18:03
@virajjasani
Copy link
Contributor

virajjasani commented Jun 10, 2025

Last 2 builds have timed out after 9 hr run, this usually happens when some tests are forever stuck.

Here also some tests are stuck it seems:

17:15:20  cd /home/jenkins/jenkins-home/workspace/enix-PreCommit-GitHub-PR_PR-2177/yetus-general-check/src
17:15:20  mvn --batch-mode -Dmaven.repo.local=/home/jenkins/jenkins-home/workspace/enix-PreCommit-GitHub-PR_PR-2177/yetus-m2/phoenix-master-patch-0 --threads=1 -DPhoenixPatchProcess -Dskip.code-coverage clean verify -fae > /home/jenkins/jenkins-home/workspace/enix-PreCommit-GitHub-PR_PR-2177/yetus-general-check/output/patch-unit-root.txt 2>&1
Cancelling nested steps due to timeout
01:48:47  Sending interrupt signal to process

@vmeka2020
Copy link
Contributor Author

vmeka2020 commented Jun 17, 2025

Build CI : https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-2177/

@sanjeet006py sanjeet006py self-requested a review June 18, 2025 15:04
sanjeet006py
sanjeet006py reviewed Jun 18, 2025
View reviewed changes
Copy link
Contributor

@sanjeet006py sanjeet006py left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM overall, left one comment. Thanks

@vmeka2020 vmeka2020 requested a review from sanjeet006py June 19, 2025 14:30
@d-c-manning
Copy link
Contributor

d-c-manning commented Jul 25, 2025

superseded by #2246?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@virajjasani virajjasani Awaiting requested review from virajjasani

@sanjeet006py sanjeet006py Awaiting requested review from sanjeet006py

1 more reviewer

@d-c-manning d-c-manning d-c-manning approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@vmeka2020 @virajjasani @d-c-manning @sanjeet006py