Skip to content

Conversation

@laanwj
Copy link
Member

@laanwj laanwj commented Apr 7, 2016

Now that #7804 fixed the timeout handling, reduce the block timeout from 20 minutes to 10 minutes. 20 minutes is overkill.

src/main.h Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Update comment please.

Copy link
Member Author

Choose a reason for hiding this comment

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

Bah, this why you never add the actual values into the comments :)
Wlll update.

Now that bitcoin#7804 fixed the timeout handling, reduce the block timeout from
20 minutes to 10 minutes. 20 minutes is overkill.
@laanwj laanwj force-pushed the 2016_04_reduce_block_timeout branch from 00bb332 to 62b9a55 Compare April 7, 2016 11:22
@btcdrak
Copy link
Contributor

btcdrak commented Apr 7, 2016

utACK

@jonasschnelli
Copy link
Contributor

utACK 62b9a55

@laanwj laanwj merged commit 62b9a55 into bitcoin:master Apr 7, 2016
laanwj added a commit that referenced this pull request Apr 7, 2016
62b9a55 Reduce block timeout to 10 minutes (Wladimir J. van der Laan)
laanwj added a commit that referenced this pull request Apr 7, 2016
Now that #7804 fixed the timeout handling, reduce the block timeout from
20 minutes to 10 minutes. 20 minutes is overkill.

Conflicts:
	src/main.h

Github-Pull: #7832
Rebased-From: 62b9a55
@maflcko
Copy link
Member

maflcko commented Jun 9, 2016

Backported as 4c3a00d.

thokon00 pushed a commit to faircoin/faircoin that referenced this pull request Jun 28, 2016
Now that bitcoin#7804 fixed the timeout handling, reduce the block timeout from
20 minutes to 10 minutes. 20 minutes is overkill.

Conflicts:
	src/main.h

Github-Pull: bitcoin#7832
Rebased-From: 62b9a55
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants