Skip to content

fix neighbours count #6409

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
Merged
merged 1 commit into from
Jul 10, 2024
Merged

fix neighbours count #6409

merged 1 commit into from
Jul 10, 2024

Conversation

vporyadke
Copy link
Collaborator

Changelog entry

...

Changelog category

  • Bugfix

Additional information

...

@github-actions github-actions bot added the bugfix label Jul 8, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 8, 2024 •
edited
Loading

⚪ 2024-07-08 13:56:19 UTC Pre-commit check for e208511 has started.
⚪ 2024-07-08 13:59:25 UTC Build linux-x86_64-release-asan is running...
🟢 2024-07-08 14:22:37 UTC Build successful.
⚪ 2024-07-08 14:22:49 UTC Tests are running...
🔴 2024-07-08 16:21:28 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
9811 9229 0 84 324 174

🟢 2024-07-08 16:22:08 UTC ydbd size 5.2 GiB changed* by -656 Bytes, which is <= 0 Bytes vs main: OK

ydbd size dash main: 6e9dd6a merge: e208511 diff diff %
ydbd size 5 605 978 616 Bytes 5 605 977 960 Bytes -656 Bytes -0.000%
ydbd stripped size 1 206 528 824 Bytes 1 206 528 888 Bytes +64 Bytes +0.000%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 8, 2024 •
edited
Loading

⚪ 2024-07-08 14:04:48 UTC Pre-commit check for e208511 has started.
⚪ 2024-07-08 14:07:24 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-07-08 14:13:34 UTC Build successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 8, 2024 •
edited
Loading

⚪ 2024-07-08 14:05:40 UTC Pre-commit check for e208511 has started.
⚪ 2024-07-08 14:08:31 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-07-08 14:46:35 UTC Build successful.
⚪ 2024-07-08 14:46:47 UTC Tests are running...
🔴 2024-07-08 16:13:16 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14085 12765 0 2 1306 12

🟢 2024-07-08 16:13:54 UTC ydbd size 8.1 GiB changed* by -1.1 KiB, which is <= 0 Bytes vs main: OK

ydbd size dash main: f94fdc3 merge: e208511 diff diff %
ydbd size 8 709 191 648 Bytes 8 709 190 504 Bytes -1.1 KiB -0.000%
ydbd stripped size 475 593 336 Bytes 475 593 080 Bytes -256 Bytes -0.000%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@vporyadke vporyadke linked an issue Jul 8, 2024 that may be closed by this pull request
Corrctly count neighbours in Hive #6424
Closed
adameat
adameat approved these changes Jul 10, 2024
View reviewed changes
Copy link
Member

@adameat adameat left a comment

Choose a reason for hiding this comment

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

also, dont forget about 24-3 (and 24-2)

@vporyadke vporyadke merged commit b81dfa9 into ydb-platform:main Jul 10, 2024
9 of 13 checks passed
This was referenced Jul 12, 2024
Merged
Merged
vporyadke added a commit to vporyadke/ydb that referenced this pull request Jul 16, 2024
@vporyadke
fix neighbours count (ydb-platform#6409)
3c37429
@vporyadke vporyadke mentioned this pull request Jul 16, 2024
Merged
@GrigoriyPA GrigoriyPA mentioned this pull request Jul 16, 2024
Merged
vporyadke added a commit that referenced this pull request Jul 17, 2024
@vporyadke
fix neighbours count (#6409) (#6718)
3aec0fb
This was referenced Jul 22, 2024
Merged
Merged
Merged
Merged
Merged
Closed
@niksaveliev niksaveliev mentioned this pull request Aug 15, 2024
Closed
This was referenced Aug 16, 2024
Closed
Merged
@StekPerepolnen StekPerepolnen mentioned this pull request Aug 23, 2024
Closed
@siarheivesialou siarheivesialou mentioned this pull request Aug 28, 2024
Closed
@swalrus1 swalrus1 mentioned this pull request Sep 3, 2024
Closed
This was referenced Oct 29, 2024
Closed
Closed
@GrigoriyPA GrigoriyPA mentioned this pull request Feb 21, 2025
Closed
@rekby rekby mentioned this pull request Jun 3, 2025
Closed
@rekby rekby mentioned this pull request Jun 3, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adameat adameat adameat approved these changes

Assignees
No one assigned
Labels
bugfix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Corrctly count neighbours in Hive
2 participants
@vporyadke @adameat