{"action":"created","comment":{"url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/pulls/comments/76852042","id":76852042,"diff_hunk":"@@ -7,9 +7,9 @@\n     </encoder>\n   </appender>\n \n-  <logger name=\"org.sagebionetworks.bridge.sdk.BaseApiCaller\" level=\"debug\" />\n+  <logger name=\"org.sagebionetworks.bridge.sdk.BaseApiCaller\" level=\"warn\" />","path":"src/test/resources/logback-test.xml","position":5,"original_position":5,"commit_id":"c68a0f3ff46f00a1f9655042fbd490d117c216e7","original_commit_id":"c68a0f3ff46f00a1f9655042fbd490d117c216e7","user":{"login":"alxdarksage","id":5589085,"avatar_url":"https://avatars.githubusercontent.com/u/5589085?v=3","gravatar_id":"","url":"https://api.github.com/users/alxdarksage","html_url":"https://github.com/alxdarksage","followers_url":"https://api.github.com/users/alxdarksage/followers","following_url":"https://api.github.com/users/alxdarksage/following{/other_user}","gists_url":"https://api.github.com/users/alxdarksage/gists{/gist_id}","starred_url":"https://api.github.com/users/alxdarksage/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alxdarksage/subscriptions","organizations_url":"https://api.github.com/users/alxdarksage/orgs","repos_url":"https://api.github.com/users/alxdarksage/repos","events_url":"https://api.github.com/users/alxdarksage/events{/privacy}","received_events_url":"https://api.github.com/users/alxdarksage/received_events","type":"User","site_admin":false},"body":"I like setting logging to debugging because when there’s an error, you can see the JSON traffic sent back and forth to the server, and it’s tedious to change and redeploy it. But also fine to change this back to warning","created_at":"2016-08-30T18:27:24Z","updated_at":"2016-08-30T18:27:24Z","html_url":"https://github.com/Sage-Bionetworks/BridgeIntegrationTests/pull/76#discussion_r76852042","pull_request_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/pulls/76","_links":{"self":{"href":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/pulls/comments/76852042"},"html":{"href":"https://github.com/Sage-Bionetworks/BridgeIntegrationTests/pull/76#discussion_r76852042"},"pull_request":{"href":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/pulls/76"}}},"pull_request":{"url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/pulls/76","id":83187168,"html_url":"https://github.com/Sage-Bionetworks/BridgeIntegrationTests/pull/76","diff_url":"https://github.com/Sage-Bionetworks/BridgeIntegrationTests/pull/76.diff","patch_url":"https://github.com/Sage-Bionetworks/BridgeIntegrationTests/pull/76.patch","issue_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/issues/76","number":76,"state":"open","locked":false,"title":"Minor schema test changes in preparation of upcoming mutable schema changes","user":{"login":"DwayneJengSage","id":9541190,"avatar_url":"https://avatars.githubusercontent.com/u/9541190?v=3","gravatar_id":"","url":"https://api.github.com/users/DwayneJengSage","html_url":"https://github.com/DwayneJengSage","followers_url":"https://api.github.com/users/DwayneJengSage/followers","following_url":"https://api.github.com/users/DwayneJengSage/following{/other_user}","gists_url":"https://api.github.com/users/DwayneJengSage/gists{/gist_id}","starred_url":"https://api.github.com/users/DwayneJengSage/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DwayneJengSage/subscriptions","organizations_url":"https://api.github.com/users/DwayneJengSage/orgs","repos_url":"https://api.github.com/users/DwayneJengSage/repos","events_url":"https://api.github.com/users/DwayneJengSage/events{/privacy}","received_events_url":"https://api.github.com/users/DwayneJengSage/received_events","type":"User","site_admin":false},"body":"There are a few minor changes in mutable schema validation. Before we push those changes server-side, we need to make some adjustments to the integ tests. In particular:\r\n\r\n* Split the schema test for \"optional values\" to create separate fields for various field properties, instead of one mega-field with all properties.\r\n\r\n* Added fields must have minAppVersion specified.\r\n\r\nThese are technically breaking changes, but given there are no automated processes that use the new field properties or update schema revisions in place, these changes should be safe to make.\r\n\r\nTesting done:\r\n* Ran against local server to verify compatibility with changes.\r\n* Ran against dev to verify compatibility with old code.","created_at":"2016-08-29T23:49:20Z","updated_at":"2016-08-30T18:27:24Z","closed_at":null,"merged_at":null,"merge_commit_sha":"efc212aabc0cb3b4980dcefa6340b8e97bd98110","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/pulls/76/commits","review_comments_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/pulls/76/comments","review_comment_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/issues/76/comments","statuses_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/statuses/c68a0f3ff46f00a1f9655042fbd490d117c216e7","head":{"label":"DwayneJengSage:develop","ref":"develop","sha":"c68a0f3ff46f00a1f9655042fbd490d117c216e7","user":{"login":"DwayneJengSage","id":9541190,"avatar_url":"https://avatars.githubusercontent.com/u/9541190?v=3","gravatar_id":"","url":"https://api.github.com/users/DwayneJengSage","html_url":"https://github.com/DwayneJengSage","followers_url":"https://api.github.com/users/DwayneJengSage/followers","following_url":"https://api.github.com/users/DwayneJengSage/following{/other_user}","gists_url":"https://api.github.com/users/DwayneJengSage/gists{/gist_id}","starred_url":"https://api.github.com/users/DwayneJengSage/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DwayneJengSage/subscriptions","organizations_url":"https://api.github.com/users/DwayneJengSage/orgs","repos_url":"https://api.github.com/users/DwayneJengSage/repos","events_url":"https://api.github.com/users/DwayneJengSage/events{/privacy}","received_events_url":"https://api.github.com/users/DwayneJengSage/received_events","type":"User","site_admin":false},"repo":{"id":53004537,"name":"BridgeIntegrationTests","full_name":"DwayneJengSage/BridgeIntegrationTests","owner":{"login":"DwayneJengSage","id":9541190,"avatar_url":"https://avatars.githubusercontent.com/u/9541190?v=3","gravatar_id":"","url":"https://api.github.com/users/DwayneJengSage","html_url":"https://github.com/DwayneJengSage","followers_url":"https://api.github.com/users/DwayneJengSage/followers","following_url":"https://api.github.com/users/DwayneJengSage/following{/other_user}","gists_url":"https://api.github.com/users/DwayneJengSage/gists{/gist_id}","starred_url":"https://api.github.com/users/DwayneJengSage/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DwayneJengSage/subscriptions","organizations_url":"https://api.github.com/users/DwayneJengSage/orgs","repos_url":"https://api.github.com/users/DwayneJengSage/repos","events_url":"https://api.github.com/users/DwayneJengSage/events{/privacy}","received_events_url":"https://api.github.com/users/DwayneJengSage/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/DwayneJengSage/BridgeIntegrationTests","description":"Integration Tests for Bridge server","fork":true,"url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests","forks_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/forks","keys_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/keys{/key_id}","collaborators_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/teams","hooks_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/hooks","issue_events_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/issues/events{/number}","events_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/events","assignees_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/assignees{/user}","branches_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/branches{/branch}","tags_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/tags","blobs_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/git/refs{/sha}","trees_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/git/trees{/sha}","statuses_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/statuses/{sha}","languages_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/languages","stargazers_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/stargazers","contributors_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/contributors","subscribers_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/subscribers","subscription_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/subscription","commits_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/commits{/sha}","git_commits_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/git/commits{/sha}","comments_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/comments{/number}","issue_comment_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/issues/comments{/number}","contents_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/contents/{+path}","compare_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/compare/{base}...{head}","merges_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/merges","archive_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/downloads","issues_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/issues{/number}","pulls_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/pulls{/number}","milestones_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/milestones{/number}","notifications_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/labels{/name}","releases_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/releases{/id}","deployments_url":"https://api.github.com/repos/DwayneJengSage/BridgeIntegrationTests/deployments","created_at":"2016-03-02T23:33:15Z","updated_at":"2016-03-03T00:43:40Z","pushed_at":"2016-08-29T23:46:29Z","git_url":"git://github.com/DwayneJengSage/BridgeIntegrationTests.git","ssh_url":"git@github.com:DwayneJengSage/BridgeIntegrationTests.git","clone_url":"https://github.com/DwayneJengSage/BridgeIntegrationTests.git","svn_url":"https://github.com/DwayneJengSage/BridgeIntegrationTests","homepage":null,"size":259,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"develop"}},"base":{"label":"Sage-Bionetworks:develop","ref":"develop","sha":"da758de067248b8f3ec1494b6f42c3128bc33ecc","user":{"login":"Sage-Bionetworks","id":1250897,"avatar_url":"https://avatars.githubusercontent.com/u/1250897?v=3","gravatar_id":"","url":"https://api.github.com/users/Sage-Bionetworks","html_url":"https://github.com/Sage-Bionetworks","followers_url":"https://api.github.com/users/Sage-Bionetworks/followers","following_url":"https://api.github.com/users/Sage-Bionetworks/following{/other_user}","gists_url":"https://api.github.com/users/Sage-Bionetworks/gists{/gist_id}","starred_url":"https://api.github.com/users/Sage-Bionetworks/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Sage-Bionetworks/subscriptions","organizations_url":"https://api.github.com/users/Sage-Bionetworks/orgs","repos_url":"https://api.github.com/users/Sage-Bionetworks/repos","events_url":"https://api.github.com/users/Sage-Bionetworks/events{/privacy}","received_events_url":"https://api.github.com/users/Sage-Bionetworks/received_events","type":"Organization","site_admin":false},"repo":{"id":52921006,"name":"BridgeIntegrationTests","full_name":"Sage-Bionetworks/BridgeIntegrationTests","owner":{"login":"Sage-Bionetworks","id":1250897,"avatar_url":"https://avatars.githubusercontent.com/u/1250897?v=3","gravatar_id":"","url":"https://api.github.com/users/Sage-Bionetworks","html_url":"https://github.com/Sage-Bionetworks","followers_url":"https://api.github.com/users/Sage-Bionetworks/followers","following_url":"https://api.github.com/users/Sage-Bionetworks/following{/other_user}","gists_url":"https://api.github.com/users/Sage-Bionetworks/gists{/gist_id}","starred_url":"https://api.github.com/users/Sage-Bionetworks/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Sage-Bionetworks/subscriptions","organizations_url":"https://api.github.com/users/Sage-Bionetworks/orgs","repos_url":"https://api.github.com/users/Sage-Bionetworks/repos","events_url":"https://api.github.com/users/Sage-Bionetworks/events{/privacy}","received_events_url":"https://api.github.com/users/Sage-Bionetworks/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Sage-Bionetworks/BridgeIntegrationTests","description":"Integration Tests for Bridge server","fork":false,"url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests","forks_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/forks","keys_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/teams","hooks_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/hooks","issue_events_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/issues/events{/number}","events_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/events","assignees_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/assignees{/user}","branches_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/branches{/branch}","tags_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/tags","blobs_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/git/refs{/sha}","trees_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/statuses/{sha}","languages_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/languages","stargazers_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/stargazers","contributors_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/contributors","subscribers_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/subscribers","subscription_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/subscription","commits_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/commits{/sha}","git_commits_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/git/commits{/sha}","comments_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/comments{/number}","issue_comment_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/issues/comments{/number}","contents_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/contents/{+path}","compare_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/merges","archive_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/downloads","issues_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/issues{/number}","pulls_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/pulls{/number}","milestones_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/milestones{/number}","notifications_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/labels{/name}","releases_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/releases{/id}","deployments_url":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/deployments","created_at":"2016-03-02T00:34:56Z","updated_at":"2016-03-03T16:41:28Z","pushed_at":"2016-08-29T23:49:20Z","git_url":"git://github.com/Sage-Bionetworks/BridgeIntegrationTests.git","ssh_url":"git@github.com:Sage-Bionetworks/BridgeIntegrationTests.git","clone_url":"https://github.com/Sage-Bionetworks/BridgeIntegrationTests.git","svn_url":"https://github.com/Sage-Bionetworks/BridgeIntegrationTests","homepage":null,"size":256,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"open_issues_count":1,"forks":3,"open_issues":1,"watchers":0,"default_branch":"develop"}},"_links":{"self":{"href":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/pulls/76"},"html":{"href":"https://github.com/Sage-Bionetworks/BridgeIntegrationTests/pull/76"},"issue":{"href":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/issues/76"},"comments":{"href":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/issues/76/comments"},"review_comments":{"href":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/pulls/76/comments"},"review_comment":{"href":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/pulls/76/commits"},"statuses":{"href":"https://api.github.com/repos/Sage-Bionetworks/BridgeIntegrationTests/statuses/c68a0f3ff46f00a1f9655042fbd490d117c216e7"}}}}
{"actor":{"display_login":"alxdarksage"}}