{"action":"created","issue":{"url":"https://api.github.com/repos/wiredtiger/wiredtiger/issues/3082","repository_url":"https://api.github.com/repos/wiredtiger/wiredtiger","labels_url":"https://api.github.com/repos/wiredtiger/wiredtiger/issues/3082/labels{/name}","comments_url":"https://api.github.com/repos/wiredtiger/wiredtiger/issues/3082/comments","events_url":"https://api.github.com/repos/wiredtiger/wiredtiger/issues/3082/events","html_url":"https://github.com/wiredtiger/wiredtiger/pull/3082","id":181214438,"number":3082,"title":"WT-2949 Add option to skip closing the connection after test runs.","user":{"login":"sueloverso","id":2945957,"avatar_url":"https://avatars.githubusercontent.com/u/2945957?v=3","gravatar_id":"","url":"https://api.github.com/users/sueloverso","html_url":"https://github.com/sueloverso","followers_url":"https://api.github.com/users/sueloverso/followers","following_url":"https://api.github.com/users/sueloverso/following{/other_user}","gists_url":"https://api.github.com/users/sueloverso/gists{/gist_id}","starred_url":"https://api.github.com/users/sueloverso/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sueloverso/subscriptions","organizations_url":"https://api.github.com/users/sueloverso/orgs","repos_url":"https://api.github.com/users/sueloverso/repos","events_url":"https://api.github.com/users/sueloverso/events{/privacy}","received_events_url":"https://api.github.com/users/sueloverso/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-10-05T17:16:56Z","updated_at":"2016-10-05T23:57:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/wiredtiger/wiredtiger/pulls/3082","html_url":"https://github.com/wiredtiger/wiredtiger/pull/3082","diff_url":"https://github.com/wiredtiger/wiredtiger/pull/3082.diff","patch_url":"https://github.com/wiredtiger/wiredtiger/pull/3082.patch"},"body":"@agorrod Please review this change to add a `close_conn` option to `wtperf`.  I default it to false so that most tests just exit.  I added the option to the config files where the workload is a populate phase and a later test is expecting to set `create=false`.\r\n\r\nIt saves 9 minutes off the `wiredtiger-perf-stress` test, from 59 down to 50 minutes.  I expect the savings to be less for the smaller and shorter test groups."},"comment":{"url":"https://api.github.com/repos/wiredtiger/wiredtiger/issues/comments/251833103","html_url":"https://github.com/wiredtiger/wiredtiger/pull/3082#issuecomment-251833103","issue_url":"https://api.github.com/repos/wiredtiger/wiredtiger/issues/3082","id":251833103,"user":{"login":"agorrod","id":1930121,"avatar_url":"https://avatars.githubusercontent.com/u/1930121?v=3","gravatar_id":"","url":"https://api.github.com/users/agorrod","html_url":"https://github.com/agorrod","followers_url":"https://api.github.com/users/agorrod/followers","following_url":"https://api.github.com/users/agorrod/following{/other_user}","gists_url":"https://api.github.com/users/agorrod/gists{/gist_id}","starred_url":"https://api.github.com/users/agorrod/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/agorrod/subscriptions","organizations_url":"https://api.github.com/users/agorrod/orgs","repos_url":"https://api.github.com/users/agorrod/repos","events_url":"https://api.github.com/users/agorrod/events{/privacy}","received_events_url":"https://api.github.com/users/agorrod/received_events","type":"User","site_admin":false},"created_at":"2016-10-05T23:57:08Z","updated_at":"2016-10-05T23:57:08Z","body":"I'm voting with @michaelcahill on this one - enabling the option default doesn't feel right.\r\n\r\n> can we go the other way and set it to false for the long-running tests where we're sure we'll never care about using the data again?\r\n\r\nWe could set it to false in the ``wtperf_run.sh`` script used by Jenkins, using ``-o close_conn=false`` as part of the wtperf command line. That shouldn't introduce any problems when reproducing Jenkins regressions locally, since the shutdown time isn't included in any of the throughput calculations."}}
{"actor":{"display_login":"agorrod"}}