Skip to content
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.

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

Add RemoteData to YQL operation progress. Add cluster_name for YT YQL operations #4648

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Krock21
Copy link
Collaborator

@Krock21 Krock21 commented May 17, 2024

Changelog entry

Changelog category

  • Not for changelog (changelog entry is not required)

Additional information

Ytsaurus issue: ytsaurus/ytsaurus#551

Previously we added yt cluster name to yql_statistics. Now we discovered that yql_statistics is populated once at the end of an operation and this is not convenient for debugging why an operation is taking too long.

Adding the same field to yql_progress which is populated during operation execution

I deployed it to our yt cluster and I see it in yql_progress now:

"remoteData": {
    "cluster_name": "dirac"
},
"remoteId": "http-proxies.dirac.svc.cluster.local/operation-id-here",

I can confirm that it becomes available during an operation execution

Copy link

github-actions bot commented May 17, 2024

2024-05-17 12:42:19 UTC Pre-commit check for a21cc90 has started.
2024-05-17 12:42:22 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-05-17 12:54:26 UTC Build successful.
2024-05-17 12:56:10 UTC Tests are running...
🔴 2024-05-17 14:45:24 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
72205 59522 0 1 12672 10

Copy link

github-actions bot commented May 17, 2024

2024-05-17 12:42:23 UTC Pre-commit check for a21cc90 has started.
2024-05-17 12:42:25 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-05-17 12:55:02 UTC Build successful.

Copy link

github-actions bot commented May 17, 2024

2024-05-17 12:42:31 UTC Pre-commit check for a21cc90 has started.
2024-05-17 12:42:33 UTC Build linux-x86_64-release-asan is running...
🟢 2024-05-17 12:55:25 UTC Build successful.
2024-05-17 12:57:11 UTC Tests are running...
🔴 2024-05-17 14:52:25 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
13374 13134 0 43 169 28

@Krock21 Krock21 added the area/yql YQL query language issues label May 31, 2024
Copy link

github-actions bot commented May 31, 2024

2024-05-31 11:31:19 UTC Pre-commit check for 25f547b has started.
2024-05-31 11:33:53 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-05-31 11:49:30 UTC Build successful.

@vitstn vitstn enabled auto-merge (squash) May 31, 2024 11:31
@vitstn vitstn disabled auto-merge May 31, 2024 11:31
Copy link

github-actions bot commented May 31, 2024

2024-05-31 12:02:13 UTC Pre-commit check for 25f547b has started.
2024-05-31 12:04:48 UTC Build linux-x86_64-release-asan is running...
🟢 2024-05-31 12:21:30 UTC Build successful.
2024-05-31 12:21:40 UTC Tests are running...
🔴 2024-05-31 14:12:13 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14545 14431 0 47 57 10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/yql YQL query language issues not-for-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants