Skip to content

Conversation

UdjinM6
Copy link

@UdjinM6 UdjinM6 commented Dec 23, 2020

Before After
Screenshot 2020-12-23 at 16 48 12 Screenshot 2020-12-23 at 16 55 32

The reason to touch dash-q-phase (even though it looks ok on screenshots above) is the way it works on regtest.
Before:

dash-q-phase-10 | RenameThread: thread new name dash-q-phase-100
dash-q-phase-10 | q-phase-100 thread start
dash-q-phase-10 | RenameThread: thread new name dash-q-phase-102
dash-q-phase-10 | q-phase-102 thread start

After:

  dash-llmq-100 | RenameThread: thread new name dash-llmq-100
  dash-llmq-100 | llmq-100 thread start
  dash-llmq-102 | RenameThread: thread new name dash-llmq-102
  dash-llmq-102 | llmq-102 thread start

Based on #3896 atm due to merge conflicts.

@UdjinM6 UdjinM6 added this to the 17 milestone Dec 23, 2020
@UdjinM6 UdjinM6 marked this pull request as ready for review December 24, 2020 01:14
@UdjinM6 UdjinM6 changed the title Fix thread names to fit into 15 symbols Change thread names to fit into 15 symbols Dec 26, 2020
@UdjinM6 UdjinM6 marked this pull request as draft December 26, 2020 02:55
@UdjinM6 UdjinM6 marked this pull request as ready for review December 27, 2020 04:30
Copy link

@xdustinface xdustinface left a comment

Choose a reason for hiding this comment

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

utACK

Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

utACK

@PastaPastaPasta PastaPastaPasta merged commit bfe6c7c into dashpay:develop Dec 30, 2020
@sidhujag
Copy link

dash-q-cachepop is ok?

@UdjinM6 UdjinM6 deleted the fix_thrnames branch July 1, 2021 21:55
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Mar 13, 2022
Duddino pushed a commit to Duddino/PIVX that referenced this pull request Jan 14, 2025
Duddino pushed a commit to Duddino/PIVX that referenced this pull request Jan 14, 2025
Duddino pushed a commit to Duddino/PIVX that referenced this pull request Jan 14, 2025
Fuzzbawls added a commit to PIVX-Project/PIVX that referenced this pull request Feb 1, 2025
595a672 Fix thread names to fit into 15 symbols (dashpay#3897) (UdjinM6)
2f492d4 evo/llmq/spork: Fix various (potential) locking issues (dashpay#3829) (UdjinM6)
453a74f Fix two potential issues in the way pending islocks are processed (dashpay#3678) (UdjinM6)
902809c replace `inter-quorum` with `intra-quorum` (dashpay#3642) (PastaPastaPasta)
46e51db Fix CSigningManager::VerifyRecoveredSig (dashpay#3546) (UdjinM6)
dd02dfd Use exponential backoff timeouts for recovery (dashpay#3535) (Alexander Block)

Pull request description:

  Each commits backports a different PR, as mentioned in the commit name

ACKs for top commit: 595a672
  Liquid369:
    ACK 595a672
  Fuzzbawls:
    utACK 595a672

Tree-SHA512: 4d66215d3b1944266802045f66d2bec1a9e4eb7c7e2bfb16fcaf16bafda7e47225b43129094081990a5a25a9b9f753e9071f30d357dbe51e592f3b824384a461
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants