💬 sdaftuar commented on pull request "Cluster mempool":
(https://github.com/bitcoin/bitcoin/pull/33629#discussion_r2538844903)
Took this wording in 34e32985e811607e7566ae7a6caeacdf8bd8384f

💬 sdaftuar commented on pull request "Cluster mempool":
(https://github.com/bitcoin/bitcoin/pull/33629#discussion_r2538847018)
Updated the comment in 47ab32fdb158069d4422e0f92078603c6df070a6.

💬 sdaftuar commented on pull request "Cluster mempool":
(https://github.com/bitcoin/bitcoin/pull/33629#discussion_r2538849747)
Took this comment and incorporated in 21b5cea588a7bfe758a8d14efe90046b111db428.

💬 sdaftuar commented on pull request "Cluster mempool":
(https://github.com/bitcoin/bitcoin/pull/33629#discussion_r2538850474)
Done.

💬 pinheadmz commented on pull request "Introduce SockMan ("lite"): low-level socket handling for HTTP":
(https://github.com/bitcoin/bitcoin/pull/32747#issuecomment-3548455862)
rebased on master to fix conflict but also converted to draft.

The plan is now to integrate this I/O loop directly with the new http server. So I will rebase #32061 on this branch to include the feedback the socket stuff has received, and then combine this sockman with the http server and remove the abstraction.

💬 sdaftuar commented on pull request "Cluster mempool":
(https://github.com/bitcoin/bitcoin/pull/33629#discussion_r2538851944)
I swapped the commits, and verified that every commit in the branch I just pushed compiles.

💬 sdaftuar commented on pull request "Cluster mempool":
(https://github.com/bitcoin/bitcoin/pull/33629#discussion_r2538853181)
Squashed.

💬 sdaftuar commented on pull request "Cluster mempool":
(https://github.com/bitcoin/bitcoin/pull/33629#discussion_r2538858278)
Fixed this commit message (8e49477e86b3089ea70d1f2659b9fd3a8a1f7db4).

💬 sdaftuar commented on pull request "Cluster mempool":
(https://github.com/bitcoin/bitcoin/pull/33629#discussion_r2538862686)
Fixed in b9a2039f51226dce2c4e38ce5f26eefee171744b.

💬 sdaftuar commented on pull request "Cluster mempool":
(https://github.com/bitcoin/bitcoin/pull/33629#discussion_r2538863183)
Squashed.

💬 sdaftuar commented on pull request "Cluster mempool":
(https://github.com/bitcoin/bitcoin/pull/33629#discussion_r2538865118)
Done.

💬 sdaftuar commented on pull request "Cluster mempool":
(https://github.com/bitcoin/bitcoin/pull/33629#discussion_r2538866223)
Thanks, fixed this wording in e031085fd464b528c186948d3cbf1c08a5a8d624

✅ willcl-ark closed a pull request: "Clear out space on GHA jobs"
(https://github.com/bitcoin/bitcoin/pull/33514)

💬 willcl-ark commented on pull request "Clear out space on GHA jobs":
(https://github.com/bitcoin/bitcoin/pull/33514#issuecomment-3548476975)
Closing for now

💬 sdaftuar commented on pull request "Cluster mempool":
(https://github.com/bitcoin/bitcoin/pull/33629#discussion_r2538870170)
I moved this commit much earlier in the PR (to right before the commit: "Select transactions for blocks based on chunk feerate").

💬 maflcko commented on pull request "Remove HD seed reference from blank wallet tooltip":
(https://github.com/bitcoin-core/gui/pull/908#issuecomment-3548479257)
lgtm ACK 929f69d0ff29cb803769a423035fdcf675f40b78

📝 kevkevinpal opened a pull request: "rest: add interface for gettxspendingprevout rpc"
(https://github.com/bitcoin/bitcoin/pull/33904)
This creates a REST interface for the `gettxspendingprevout` RPC, it also includes functional tests and updates to the REST docs

This is part of https://github.com/bitcoin/bitcoin/issues/33808 but would still need [@sstone’s](https://github.com/sstone) PR https://github.com/bitcoin/bitcoin/pull/24539, but that can be left as a follow-up for when it is merged.

If it is prefered to open a PR after https://github.com/bitcoin/bitcoin/issues/33808 I can close this and reopen after its merged

💬 vasild commented on pull request "Broadcast own transactions only via short-lived Tor or I2P connections":
(https://github.com/bitcoin/bitcoin/pull/29415#issuecomment-3548620601)
`3e6b5a9ec4...b499edb414`: take https://github.com/bitcoin/bitcoin/pull/29415#discussion_r2527749355

💬 instagibbs commented on issue "test: intermittent issue in p2p_1p1c_network.py":
(https://github.com/bitcoin/bitcoin/issues/33318#issuecomment-3548640184)
I've also discovered that `create_package_2outs` has regressed since we lowered minrelay to 0.1s/vbyte. The `parent_4` now pays for itself rather than needing cppf.

I think it'd be better if we do 0-fee parents to make the test less brittle. I have a branch that I can push along later shortly after cluster mempool and https://github.com/bitcoin/bitcoin/pull/33892 is merged: https://github.com/instagibbs/bitcoin/commits/2025-11-1p1c-test-timeout/

As per branch I think just making sure the orpha
...

💬 vasild commented on pull request "Broadcast own transactions only via short-lived Tor or I2P connections":
(https://github.com/bitcoin/bitcoin/pull/29415#discussion_r2539016880)
Yeah, `compare_exchange_weak()` would set the current value into its first argument if it fails. No need to assign it again explicitly. Changed.

Some napkin math about the overflow - at 7 transactions per second, to have 2³² would take about 20 years. Further, `m_num_to_open` is about unconfirmed transactions, connections that the node needs to open because it has that many transactions to send. 100 bytes each means that the node stores 400GB of transactions to be broadcast. Can't
...

💬 willcl-ark commented on issue "ci: windows-native-dll-vcpkg-* cache does not work?":
(https://github.com/bitcoin/bitcoin/issues/33685#issuecomment-3548778624)
I checked the last 200 runs to see if this was still happening, and it does seem to, but a lot less frequently than when this was opened.

<details>
<summary>Details</summary>

```json
{
"job_name": "Windows native, VS 2022",
"step_name": "Generate build system",
"repo": "bitcoin/bitcoin",
"duration_seconds": [
191.0,
100.0,
101.0,
91.0,
91.0,
88.0,
110.0,
163.0,
115.0,
87.0,
115.0,
92.0,
87.0,
110.0,
98.0,
91.0,
96.0,

...