{"action":"created","issue":{"url":"https://api.github.com/repos/jenkinsci/durable-task-plugin/issues/84","repository_url":"https://api.github.com/repos/jenkinsci/durable-task-plugin","labels_url":"https://api.github.com/repos/jenkinsci/durable-task-plugin/issues/84/labels{/name}","comments_url":"https://api.github.com/repos/jenkinsci/durable-task-plugin/issues/84/comments","events_url":"https://api.github.com/repos/jenkinsci/durable-task-plugin/issues/84/events","html_url":"https://github.com/jenkinsci/durable-task-plugin/pull/84","id":371778366,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI0MTQ5Njk5","number":84,"title":"Use the standard Timer rather than our own ScheduledExecutorService","user":{"login":"jglick","id":154109,"node_id":"MDQ6VXNlcjE1NDEwOQ==","avatar_url":"https://avatars1.githubusercontent.com/u/154109?v=4","gravatar_id":"","url":"https://api.github.com/users/jglick","html_url":"https://github.com/jglick","followers_url":"https://api.github.com/users/jglick/followers","following_url":"https://api.github.com/users/jglick/following{/other_user}","gists_url":"https://api.github.com/users/jglick/gists{/gist_id}","starred_url":"https://api.github.com/users/jglick/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jglick/subscriptions","organizations_url":"https://api.github.com/users/jglick/orgs","repos_url":"https://api.github.com/users/jglick/repos","events_url":"https://api.github.com/users/jglick/events{/privacy}","received_events_url":"https://api.github.com/users/jglick/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-10-19T02:01:50Z","updated_at":"2018-10-26T13:35:54Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/jenkinsci/durable-task-plugin/pulls/84","html_url":"https://github.com/jenkinsci/durable-task-plugin/pull/84","diff_url":"https://github.com/jenkinsci/durable-task-plugin/pull/84.diff","patch_url":"https://github.com/jenkinsci/durable-task-plugin/pull/84.patch"},"body":"Analogous to https://github.com/jenkinsci/workflow-durable-task-step-plugin/pull/81 but this time reverting a bit of complexity in #60. Together they seem to solve the `RestartableJenkinsRule` problems in https://github.com/jenkinsci/ant-plugin/pull/32. `Timer` should work fine in agent JVMs (it does not reference Jenkins model classes); the difference is for durable tasks (in watch mode) run on the master node: `Timer` gets properly shut down when Jenkins exits, and is thus safe for in-process restart tests. Again unlikely to have any meaningful impact in production."},"comment":{"url":"https://api.github.com/repos/jenkinsci/durable-task-plugin/issues/comments/433409805","html_url":"https://github.com/jenkinsci/durable-task-plugin/pull/84#issuecomment-433409805","issue_url":"https://api.github.com/repos/jenkinsci/durable-task-plugin/issues/84","id":433409805,"node_id":"MDEyOklzc3VlQ29tbWVudDQzMzQwOTgwNQ==","user":{"login":"jglick","id":154109,"node_id":"MDQ6VXNlcjE1NDEwOQ==","avatar_url":"https://avatars1.githubusercontent.com/u/154109?v=4","gravatar_id":"","url":"https://api.github.com/users/jglick","html_url":"https://github.com/jglick","followers_url":"https://api.github.com/users/jglick/followers","following_url":"https://api.github.com/users/jglick/following{/other_user}","gists_url":"https://api.github.com/users/jglick/gists{/gist_id}","starred_url":"https://api.github.com/users/jglick/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jglick/subscriptions","organizations_url":"https://api.github.com/users/jglick/orgs","repos_url":"https://api.github.com/users/jglick/repos","events_url":"https://api.github.com/users/jglick/events{/privacy}","received_events_url":"https://api.github.com/users/jglick/received_events","type":"User","site_admin":false},"created_at":"2018-10-26T13:35:54Z","updated_at":"2018-10-26T13:35:54Z","author_association":"MEMBER","body":"> I'd hesitate to trim back the thread pools that we have created to protect as Bulkheads to protect the (heavily used) Jenkins Timer thread pool from misbehaving Pipeline work\r\n\r\nI think you misunderstand. This is watch-mode code, running in the agent JVM. (Assuming you are running the `node` block on an agent. If you are not, who cares about scalability?) So this is not the same `Timer` pool."}}
{"actor":{"display_login":"jglick"}}