{"action":"created","comment":{"url":"https://api.github.com/repos/bitcoin/bitcoin/pulls/comments/41465323","id":41465323,"diff_hunk":"@@ -44,13 +44,13 @@ static const bool DEFAULT_ALERTS = true;\n /** Default for -maxorphantx, maximum number of orphan transactions kept in memory */\n static const unsigned int DEFAULT_MAX_ORPHAN_TRANSACTIONS = 100;\n /** Default for -limitancestorcount, max number of in-mempool ancestors */\n-static const unsigned int DEFAULT_ANCESTOR_LIMIT = 100;\n+static const unsigned int DEFAULT_ANCESTOR_LIMIT = 25;\n /** Default for -limitancestorsize, maximum kilobytes of tx + all in-mempool ancestors */\n-static const unsigned int DEFAULT_ANCESTOR_SIZE_LIMIT = 900;\n+static const unsigned int DEFAULT_ANCESTOR_SIZE_LIMIT = 100;","path":"src/main.h","position":8,"original_position":8,"commit_id":"6362ed4c4835d583f7c5fbe003d64d791c2bd159","original_commit_id":"6362ed4c4835d583f7c5fbe003d64d791c2bd159","user":{"login":"TheBlueMatt","id":649246,"avatar_url":"https://avatars.githubusercontent.com/u/649246?v=3","gravatar_id":"","url":"https://api.github.com/users/TheBlueMatt","html_url":"https://github.com/TheBlueMatt","followers_url":"https://api.github.com/users/TheBlueMatt/followers","following_url":"https://api.github.com/users/TheBlueMatt/following{/other_user}","gists_url":"https://api.github.com/users/TheBlueMatt/gists{/gist_id}","starred_url":"https://api.github.com/users/TheBlueMatt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TheBlueMatt/subscriptions","organizations_url":"https://api.github.com/users/TheBlueMatt/orgs","repos_url":"https://api.github.com/users/TheBlueMatt/repos","events_url":"https://api.github.com/users/TheBlueMatt/events{/privacy}","received_events_url":"https://api.github.com/users/TheBlueMatt/received_events","type":"User","site_admin":false},"body":"I would agree with @petertodd here, we should set the size limit to something like 101 or 105.","created_at":"2015-10-08T00:32:47Z","updated_at":"2015-10-08T00:32:47Z","html_url":"https://github.com/bitcoin/bitcoin/pull/6771#discussion_r41465323","pull_request_url":"https://api.github.com/repos/bitcoin/bitcoin/pulls/6771","_links":{"self":{"href":"https://api.github.com/repos/bitcoin/bitcoin/pulls/comments/41465323"},"html":{"href":"https://github.com/bitcoin/bitcoin/pull/6771#discussion_r41465323"},"pull_request":{"href":"https://api.github.com/repos/bitcoin/bitcoin/pulls/6771"}}},"pull_request":{"url":"https://api.github.com/repos/bitcoin/bitcoin/pulls/6771","id":46924534,"html_url":"https://github.com/bitcoin/bitcoin/pull/6771","diff_url":"https://github.com/bitcoin/bitcoin/pull/6771.diff","patch_url":"https://github.com/bitcoin/bitcoin/pull/6771.patch","issue_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/6771","number":6771,"state":"open","locked":false,"title":"Policy: Lower default limits for tx chains","user":{"login":"morcos","id":4360349,"avatar_url":"https://avatars.githubusercontent.com/u/4360349?v=3","gravatar_id":"","url":"https://api.github.com/users/morcos","html_url":"https://github.com/morcos","followers_url":"https://api.github.com/users/morcos/followers","following_url":"https://api.github.com/users/morcos/following{/other_user}","gists_url":"https://api.github.com/users/morcos/gists{/gist_id}","starred_url":"https://api.github.com/users/morcos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/morcos/subscriptions","organizations_url":"https://api.github.com/users/morcos/orgs","repos_url":"https://api.github.com/users/morcos/repos","events_url":"https://api.github.com/users/morcos/events{/privacy}","received_events_url":"https://api.github.com/users/morcos/received_events","type":"User","site_admin":false},"body":"Reduce the default limits on maximum number of transactions and the cumulative size of those transactions in both ancestor and descendant packages to 25 txs and 100kb total size.\r\n\r\nPlease see [email to bitcoin-dev](https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2015-October/011401.html) for background.\r\n\r\nI looked at some recent data and 2.6% of transactions in my node's mempool in the first 5 days of October would have been restricted by these new lower limits.","created_at":"2015-10-06T16:18:30Z","updated_at":"2015-10-08T00:32:47Z","closed_at":null,"merged_at":null,"merge_commit_sha":"4b9197a254ee17c26cb72f69ee169a0fa37faf00","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/bitcoin/bitcoin/pulls/6771/commits","review_comments_url":"https://api.github.com/repos/bitcoin/bitcoin/pulls/6771/comments","review_comment_url":"https://api.github.com/repos/bitcoin/bitcoin/pulls/comments{/number}","comments_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/6771/comments","statuses_url":"https://api.github.com/repos/bitcoin/bitcoin/statuses/6362ed4c4835d583f7c5fbe003d64d791c2bd159","head":{"label":"morcos:lowerLimits","ref":"lowerLimits","sha":"6362ed4c4835d583f7c5fbe003d64d791c2bd159","user":{"login":"morcos","id":4360349,"avatar_url":"https://avatars.githubusercontent.com/u/4360349?v=3","gravatar_id":"","url":"https://api.github.com/users/morcos","html_url":"https://github.com/morcos","followers_url":"https://api.github.com/users/morcos/followers","following_url":"https://api.github.com/users/morcos/following{/other_user}","gists_url":"https://api.github.com/users/morcos/gists{/gist_id}","starred_url":"https://api.github.com/users/morcos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/morcos/subscriptions","organizations_url":"https://api.github.com/users/morcos/orgs","repos_url":"https://api.github.com/users/morcos/repos","events_url":"https://api.github.com/users/morcos/events{/privacy}","received_events_url":"https://api.github.com/users/morcos/received_events","type":"User","site_admin":false},"repo":{"id":21699686,"name":"bitcoin","full_name":"morcos/bitcoin","owner":{"login":"morcos","id":4360349,"avatar_url":"https://avatars.githubusercontent.com/u/4360349?v=3","gravatar_id":"","url":"https://api.github.com/users/morcos","html_url":"https://github.com/morcos","followers_url":"https://api.github.com/users/morcos/followers","following_url":"https://api.github.com/users/morcos/following{/other_user}","gists_url":"https://api.github.com/users/morcos/gists{/gist_id}","starred_url":"https://api.github.com/users/morcos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/morcos/subscriptions","organizations_url":"https://api.github.com/users/morcos/orgs","repos_url":"https://api.github.com/users/morcos/repos","events_url":"https://api.github.com/users/morcos/events{/privacy}","received_events_url":"https://api.github.com/users/morcos/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/morcos/bitcoin","description":"Bitcoin Core integration/staging tree","fork":true,"url":"https://api.github.com/repos/morcos/bitcoin","forks_url":"https://api.github.com/repos/morcos/bitcoin/forks","keys_url":"https://api.github.com/repos/morcos/bitcoin/keys{/key_id}","collaborators_url":"https://api.github.com/repos/morcos/bitcoin/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/morcos/bitcoin/teams","hooks_url":"https://api.github.com/repos/morcos/bitcoin/hooks","issue_events_url":"https://api.github.com/repos/morcos/bitcoin/issues/events{/number}","events_url":"https://api.github.com/repos/morcos/bitcoin/events","assignees_url":"https://api.github.com/repos/morcos/bitcoin/assignees{/user}","branches_url":"https://api.github.com/repos/morcos/bitcoin/branches{/branch}","tags_url":"https://api.github.com/repos/morcos/bitcoin/tags","blobs_url":"https://api.github.com/repos/morcos/bitcoin/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/morcos/bitcoin/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/morcos/bitcoin/git/refs{/sha}","trees_url":"https://api.github.com/repos/morcos/bitcoin/git/trees{/sha}","statuses_url":"https://api.github.com/repos/morcos/bitcoin/statuses/{sha}","languages_url":"https://api.github.com/repos/morcos/bitcoin/languages","stargazers_url":"https://api.github.com/repos/morcos/bitcoin/stargazers","contributors_url":"https://api.github.com/repos/morcos/bitcoin/contributors","subscribers_url":"https://api.github.com/repos/morcos/bitcoin/subscribers","subscription_url":"https://api.github.com/repos/morcos/bitcoin/subscription","commits_url":"https://api.github.com/repos/morcos/bitcoin/commits{/sha}","git_commits_url":"https://api.github.com/repos/morcos/bitcoin/git/commits{/sha}","comments_url":"https://api.github.com/repos/morcos/bitcoin/comments{/number}","issue_comment_url":"https://api.github.com/repos/morcos/bitcoin/issues/comments{/number}","contents_url":"https://api.github.com/repos/morcos/bitcoin/contents/{+path}","compare_url":"https://api.github.com/repos/morcos/bitcoin/compare/{base}...{head}","merges_url":"https://api.github.com/repos/morcos/bitcoin/merges","archive_url":"https://api.github.com/repos/morcos/bitcoin/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/morcos/bitcoin/downloads","issues_url":"https://api.github.com/repos/morcos/bitcoin/issues{/number}","pulls_url":"https://api.github.com/repos/morcos/bitcoin/pulls{/number}","milestones_url":"https://api.github.com/repos/morcos/bitcoin/milestones{/number}","notifications_url":"https://api.github.com/repos/morcos/bitcoin/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/morcos/bitcoin/labels{/name}","releases_url":"https://api.github.com/repos/morcos/bitcoin/releases{/id}","created_at":"2014-07-10T15:39:10Z","updated_at":"2015-08-20T17:55:30Z","pushed_at":"2015-10-06T02:17:12Z","git_url":"git://github.com/morcos/bitcoin.git","ssh_url":"git@github.com:morcos/bitcoin.git","clone_url":"https://github.com/morcos/bitcoin.git","svn_url":"https://github.com/morcos/bitcoin","homepage":"https://bitcoin.org/en/download","size":56474,"stargazers_count":0,"watchers_count":0,"language":"C++","has_issues":false,"has_downloads":false,"has_wiki":false,"has_pages":false,"forks_count":1,"mirror_url":null,"open_issues_count":0,"forks":1,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"bitcoin:master","ref":"master","sha":"4fac576c619f450da8ca09ac8790063b3dea4364","user":{"login":"bitcoin","id":528860,"avatar_url":"https://avatars.githubusercontent.com/u/528860?v=3","gravatar_id":"","url":"https://api.github.com/users/bitcoin","html_url":"https://github.com/bitcoin","followers_url":"https://api.github.com/users/bitcoin/followers","following_url":"https://api.github.com/users/bitcoin/following{/other_user}","gists_url":"https://api.github.com/users/bitcoin/gists{/gist_id}","starred_url":"https://api.github.com/users/bitcoin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bitcoin/subscriptions","organizations_url":"https://api.github.com/users/bitcoin/orgs","repos_url":"https://api.github.com/users/bitcoin/repos","events_url":"https://api.github.com/users/bitcoin/events{/privacy}","received_events_url":"https://api.github.com/users/bitcoin/received_events","type":"Organization","site_admin":false},"repo":{"id":1181927,"name":"bitcoin","full_name":"bitcoin/bitcoin","owner":{"login":"bitcoin","id":528860,"avatar_url":"https://avatars.githubusercontent.com/u/528860?v=3","gravatar_id":"","url":"https://api.github.com/users/bitcoin","html_url":"https://github.com/bitcoin","followers_url":"https://api.github.com/users/bitcoin/followers","following_url":"https://api.github.com/users/bitcoin/following{/other_user}","gists_url":"https://api.github.com/users/bitcoin/gists{/gist_id}","starred_url":"https://api.github.com/users/bitcoin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bitcoin/subscriptions","organizations_url":"https://api.github.com/users/bitcoin/orgs","repos_url":"https://api.github.com/users/bitcoin/repos","events_url":"https://api.github.com/users/bitcoin/events{/privacy}","received_events_url":"https://api.github.com/users/bitcoin/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/bitcoin/bitcoin","description":"Bitcoin Core integration/staging tree","fork":false,"url":"https://api.github.com/repos/bitcoin/bitcoin","forks_url":"https://api.github.com/repos/bitcoin/bitcoin/forks","keys_url":"https://api.github.com/repos/bitcoin/bitcoin/keys{/key_id}","collaborators_url":"https://api.github.com/repos/bitcoin/bitcoin/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/bitcoin/bitcoin/teams","hooks_url":"https://api.github.com/repos/bitcoin/bitcoin/hooks","issue_events_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/events{/number}","events_url":"https://api.github.com/repos/bitcoin/bitcoin/events","assignees_url":"https://api.github.com/repos/bitcoin/bitcoin/assignees{/user}","branches_url":"https://api.github.com/repos/bitcoin/bitcoin/branches{/branch}","tags_url":"https://api.github.com/repos/bitcoin/bitcoin/tags","blobs_url":"https://api.github.com/repos/bitcoin/bitcoin/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/bitcoin/bitcoin/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/bitcoin/bitcoin/git/refs{/sha}","trees_url":"https://api.github.com/repos/bitcoin/bitcoin/git/trees{/sha}","statuses_url":"https://api.github.com/repos/bitcoin/bitcoin/statuses/{sha}","languages_url":"https://api.github.com/repos/bitcoin/bitcoin/languages","stargazers_url":"https://api.github.com/repos/bitcoin/bitcoin/stargazers","contributors_url":"https://api.github.com/repos/bitcoin/bitcoin/contributors","subscribers_url":"https://api.github.com/repos/bitcoin/bitcoin/subscribers","subscription_url":"https://api.github.com/repos/bitcoin/bitcoin/subscription","commits_url":"https://api.github.com/repos/bitcoin/bitcoin/commits{/sha}","git_commits_url":"https://api.github.com/repos/bitcoin/bitcoin/git/commits{/sha}","comments_url":"https://api.github.com/repos/bitcoin/bitcoin/comments{/number}","issue_comment_url":"https://api.github.com/repos/bitcoin/bitcoin/issues/comments{/number}","contents_url":"https://api.github.com/repos/bitcoin/bitcoin/contents/{+path}","compare_url":"https://api.github.com/repos/bitcoin/bitcoin/compare/{base}...{head}","merges_url":"https://api.github.com/repos/bitcoin/bitcoin/merges","archive_url":"https://api.github.com/repos/bitcoin/bitcoin/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/bitcoin/bitcoin/downloads","issues_url":"https://api.github.com/repos/bitcoin/bitcoin/issues{/number}","pulls_url":"https://api.github.com/repos/bitcoin/bitcoin/pulls{/number}","milestones_url":"https://api.github.com/repos/bitcoin/bitcoin/milestones{/number}","notifications_url":"https://api.github.com/repos/bitcoin/bitcoin/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/bitcoin/bitcoin/labels{/name}","releases_url":"https://api.github.com/repos/bitcoin/bitcoin/releases{/id}","created_at":"2010-12-19T15:16:43Z","updated_at":"2015-10-07T21:45:30Z","pushed_at":"2015-10-08T00:16:37Z","git_url":"git://github.com/bitcoin/bitcoin.git","ssh_url":"git@github.com:bitcoin/bitcoin.git","clone_url":"https://github.com/bitcoin/bitcoin.git","svn_url":"https://github.com/bitcoin/bitcoin","homepage":"https://bitcoin.org/en/download","size":136198,"stargazers_count":7605,"watchers_count":7605,"language":"C++","has_issues":true,"has_downloads":false,"has_wiki":false,"has_pages":false,"forks_count":5251,"mirror_url":null,"open_issues_count":434,"forks":5251,"open_issues":434,"watchers":7605,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/bitcoin/bitcoin/pulls/6771"},"html":{"href":"https://github.com/bitcoin/bitcoin/pull/6771"},"issue":{"href":"https://api.github.com/repos/bitcoin/bitcoin/issues/6771"},"comments":{"href":"https://api.github.com/repos/bitcoin/bitcoin/issues/6771/comments"},"review_comments":{"href":"https://api.github.com/repos/bitcoin/bitcoin/pulls/6771/comments"},"review_comment":{"href":"https://api.github.com/repos/bitcoin/bitcoin/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/bitcoin/bitcoin/pulls/6771/commits"},"statuses":{"href":"https://api.github.com/repos/bitcoin/bitcoin/statuses/6362ed4c4835d583f7c5fbe003d64d791c2bd159"}}}}