{"action":"created","issue":{"url":"https://api.github.com/repos/gearpump/gearpump/issues/1700","labels_url":"https://api.github.com/repos/gearpump/gearpump/issues/1700/labels{/name}","comments_url":"https://api.github.com/repos/gearpump/gearpump/issues/1700/comments","events_url":"https://api.github.com/repos/gearpump/gearpump/issues/1700/events","html_url":"https://github.com/gearpump/gearpump/pull/1700","id":121945729,"number":1700,"title":"fix #1670, regression, change taskActor back to use stashing.","user":{"login":"clockfly","id":2595532,"avatar_url":"https://avatars.githubusercontent.com/u/2595532?v=3","gravatar_id":"","url":"https://api.github.com/users/clockfly","html_url":"https://github.com/clockfly","followers_url":"https://api.github.com/users/clockfly/followers","following_url":"https://api.github.com/users/clockfly/following{/other_user}","gists_url":"https://api.github.com/users/clockfly/gists{/gist_id}","starred_url":"https://api.github.com/users/clockfly/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/clockfly/subscriptions","organizations_url":"https://api.github.com/users/clockfly/orgs","repos_url":"https://api.github.com/users/clockfly/repos","events_url":"https://api.github.com/users/clockfly/events{/privacy}","received_events_url":"https://api.github.com/users/clockfly/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-12-13T21:40:18Z","updated_at":"2015-12-14T07:34:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/gearpump/gearpump/pulls/1700","html_url":"https://github.com/gearpump/gearpump/pull/1700","diff_url":"https://github.com/gearpump/gearpump/pull/1700.diff","patch_url":"https://github.com/gearpump/gearpump/pull/1700.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/gearpump/gearpump/issues/comments/164368918","html_url":"https://github.com/gearpump/gearpump/pull/1700#issuecomment-164368918","issue_url":"https://api.github.com/repos/gearpump/gearpump/issues/1700","id":164368918,"user":{"login":"manuzhang","id":1191767,"avatar_url":"https://avatars.githubusercontent.com/u/1191767?v=3","gravatar_id":"","url":"https://api.github.com/users/manuzhang","html_url":"https://github.com/manuzhang","followers_url":"https://api.github.com/users/manuzhang/followers","following_url":"https://api.github.com/users/manuzhang/following{/other_user}","gists_url":"https://api.github.com/users/manuzhang/gists{/gist_id}","starred_url":"https://api.github.com/users/manuzhang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/manuzhang/subscriptions","organizations_url":"https://api.github.com/users/manuzhang/orgs","repos_url":"https://api.github.com/users/manuzhang/repos","events_url":"https://api.github.com/users/manuzhang/events{/privacy}","received_events_url":"https://api.github.com/users/manuzhang/received_events","type":"User","site_admin":false},"created_at":"2015-12-14T07:34:53Z","updated_at":"2015-12-14T07:34:53Z","body":"why do we send `ChangeTask` message to the `TaskActor` being modified ? It seems nothing is changed here \r\n\r\n```scala\r\n    case ChangeTask(_, dagVersion, life, subscribers) =>\r\n      this.life = life\r\n      subscribers.foreach { subscriber =>\r\n        val processorId = subscriber.processorId\r\n        val subscription = getSubscription(processorId)\r\n        subscription match {\r\n          case Some(subscription) =>\r\n            subscription.changeLife(subscriber.lifeTime cross this.life)\r\n          case None =>\r\n            val subscription = new Subscription(appId, executorId, taskId, subscriber, sessionId, this,\r\n              maxPendingMessageCount, ackOnceEveryMessageCount)\r\n            subscription.start\r\n            subscriptions :+= (subscriber.processorId, subscription)\r\n            // sort, keep the order\r\n            subscriptions = subscriptions.sortBy(_._1)\r\n        }\r\n      }\r\n      sender ! TaskChanged(taskId, dagVersion)\r\n```"}}