{ "id": "135214", "key": "ALOY-1116", "fields": { "issuetype": { "id": "4", "description": "An improvement or enhancement to an existing feature or task.", "name": "Improvement", "subtask": false }, "project": { "id": "11113", "key": "ALOY", "name": "Alloy", "projectCategory": { "id": "10400", "description": "Tools for developing applications", "name": "Tooling" } }, "fixVersions": [ { "id": "16692", "name": "Alloy 1.7.0", "archived": false, "released": true, "releaseDate": "2015-07-22" } ], "resolution": { "id": "1", "description": "A fix for this issue is checked into the tree and tested.", "name": "Fixed" }, "resolutiondate": "2014-09-17T15:28:57.000+0000", "created": "2014-08-22T19:01:45.000+0000", "priority": { "name": "Medium", "id": "3" }, "labels": [], "versions": [], "issuelinks": [], "assignee": { "name": "skypanther", "key": "skypanther", "displayName": "Tim Poulsen", "active": true, "timeZone": "America/New_York" }, "updated": "2014-10-23T19:47:54.000+0000", "status": { "description": "A resolution has been taken, and it is awaiting verification by reporter. From here issues are either reopened, or are closed.", "name": "Resolved", "id": "5", "statusCategory": { "id": 3, "key": "done", "colorName": "green", "name": "Done" } }, "components": [ { "id": "13605", "name": "Models" } ], "description": "Community PR: https://github.com/appcelerator/alloy/pull/518\r\n\r\nThe proposed change should offer speed improvements but implements no functional changes (in other words, existing code should continue to work).\r\n\r\nPer the comment on the PR: Consider adding a batch update method that would be faster because it would process all model changes in a single SQL statement rather than by opening repeated connections to the database. (Likely that's a separate ticket)", "attachment": [], "flagged": false, "summary": "sql sync adapter: optimize update method for adding/updating models", "creator": { "name": "skypanther", "key": "skypanther", "displayName": "Tim Poulsen", "active": true, "timeZone": "America/New_York" }, "subtasks": [], "reporter": { "name": "skypanther", "key": "skypanther", "displayName": "Tim Poulsen", "active": true, "timeZone": "America/New_York" }, "environment": null, "closedSprints": [ { "id": 217, "state": "closed", "name": "2014 Sprint 19 Alloy", "startDate": "2014-09-15T12:45:43.956Z", "endDate": "2014-09-29T12:45:00.000Z", "completeDate": "2014-09-29T13:30:39.342Z", "originBoardId": 124 }, { "id": 206, "state": "closed", "name": "2014 Sprint 18 Alloy", "startDate": "2014-09-02T13:23:09.065Z", "endDate": "2014-09-16T13:23:00.000Z", "completeDate": "2014-09-15T12:25:31.379Z", "originBoardId": 124 } ], "comment": { "comments": [ { "id": "323146", "author": { "name": "skypanther", "key": "skypanther", "displayName": "Tim Poulsen", "active": true, "timeZone": "America/New_York" }, "body": "Tested the PR. Small typo (see comment on PR) but other than that it works and passes all tests. I see no obvious side-effects and I'm ready to merge, pending a fix for the typo.", "updateAuthor": { "name": "skypanther", "key": "skypanther", "displayName": "Tim Poulsen", "active": true, "timeZone": "America/New_York" }, "created": "2014-09-11T20:00:58.000+0000", "updated": "2014-09-11T20:00:58.000+0000" }, { "id": "323295", "author": { "name": "skypanther", "key": "skypanther", "displayName": "Tim Poulsen", "active": true, "timeZone": "America/New_York" }, "body": "PR: https://github.com/appcelerator/alloy/pull/556\r\n\r\nFunctional test: any of the test/apps/models/* sample apps should run without error. ", "updateAuthor": { "name": "skypanther", "key": "skypanther", "displayName": "Tim Poulsen", "active": true, "timeZone": "America/New_York" }, "created": "2014-09-12T15:32:28.000+0000", "updated": "2014-09-12T15:32:28.000+0000" }, { "id": "323573", "author": { "name": "fmiao", "key": "fmiao", "displayName": "Feon Sua Xin Miao", "active": true, "timeZone": "America/Vancouver" }, "body": "PR merged.", "updateAuthor": { "name": "fmiao", "key": "fmiao", "displayName": "Feon Sua Xin Miao", "active": true, "timeZone": "America/Vancouver" }, "created": "2014-09-15T04:16:17.000+0000", "updated": "2014-09-15T04:16:17.000+0000" }, { "id": "323620", "author": { "name": "skypanther", "key": "skypanther", "displayName": "Tim Poulsen", "active": true, "timeZone": "America/New_York" }, "body": "PR to back out the change https://github.com/appcelerator/alloy/pull/562\r\n\r\nThe immediately-invoked function is necessary to permit garbage collection", "updateAuthor": { "name": "skypanther", "key": "skypanther", "displayName": "Tim Poulsen", "active": true, "timeZone": "America/New_York" }, "created": "2014-09-15T12:04:10.000+0000", "updated": "2014-09-15T12:04:10.000+0000" }, { "id": "323965", "author": { "name": "skypanther", "key": "skypanther", "displayName": "Tim Poulsen", "active": true, "timeZone": "America/New_York" }, "body": "New PR https://github.com/appcelerator/alloy/pull/564\r\n\r\nThis change restores the self-calling (IIFE) function, removes the unnecessary transaction, and implements the lastInsertId as in the original community PR. Please merge pull 562 before merging this one.", "updateAuthor": { "name": "skypanther", "key": "skypanther", "displayName": "Tim Poulsen", "active": true, "timeZone": "America/New_York" }, "created": "2014-09-16T15:16:16.000+0000", "updated": "2014-09-16T15:16:16.000+0000" }, { "id": "324238", "author": { "name": "fmiao", "key": "fmiao", "displayName": "Feon Sua Xin Miao", "active": true, "timeZone": "America/Vancouver" }, "body": "PR merged.", "updateAuthor": { "name": "fmiao", "key": "fmiao", "displayName": "Feon Sua Xin Miao", "active": true, "timeZone": "America/Vancouver" }, "created": "2014-09-17T15:28:57.000+0000", "updated": "2014-09-17T15:28:57.000+0000" } ], "maxResults": 6, "total": 6, "startAt": 0 } } }