{"action":"created","comment":{"url":"https://api.github.com/repos/Azure/azure-quickstart-templates/pulls/comments/82677176","pull_request_review_id":3568504,"id":82677176,"diff_hunk":"@@ -1,89 +1,94 @@\n   {\n     \"contentVersion\": \"1.0.0.0\",\n-    \"$schema\": \"http://schema.management.azure.com/schemas/2015-01-01/deploymentTemplate.json#\", \n+    \"$schema\": \"http://schema.management.azure.com/schemas/2015-01-01/deploymentTemplate.json#\",\n     \"parameters\": {\n       \"dataFactoryName\": {\n         \"type\": \"string\",\n         \"metadata\": {\n-          \"description\":  \"Data factory name\"\n+          \"description\": \"Data factory name\"\n         }\n       },\n       \"storageAccountName\": {\n         \"type\": \"string\",\n         \"metadata\": {\n-          \"description\":  \"Azure Storage account name\"\n+          \"description\": \"Name of the Azure storage account that contains the data to be copied.\"\n         }\n       },\n       \"storageAccountKey\": {\n         \"type\": \"securestring\",\n         \"metadata\": {\n-          \"description\":  \"Azure Storage account key\"\n+          \"description\": \"Key for the Azure storage account.\"\n         }\n       },\n       \"sourceBlobContainer\": {\n         \"type\": \"string\",\n         \"metadata\": {\n-          \"description\":  \"Azure blob container name\"\n+          \"description\": \"Name of the blob container in the Azure Storage account.\"\n         }\n       },\n       \"sourceBlobName\": {\n         \"type\": \"string\",\n         \"metadata\": {\n-          \"description\":  \"Azure blob name\"\n+          \"description\": \"Name of the blob in the container that has the data to be copied to Azure SQL Database table\"\n         }\n       },\n       \"sqlServerName\": {\n         \"type\": \"string\",\n         \"metadata\": {\n-          \"description\":  \"Azure SQL server name\"\n+          \"description\": \"Name of the Azure SQL Server that will hold the output/copied data.\"\n         }\n       },\n       \"databaseName\": {\n         \"type\": \"string\",\n         \"metadata\": {\n-          \"description\":  \"Azure SQL database name\"\n+          \"description\": \"Name of the Azure SQL Database in the Azure SQL server.\"\n         }\n       },\n       \"sqlServerUserName\": {\n         \"type\": \"string\",\n         \"metadata\": {\n-          \"description\":  \"Azure SQL server - user name\"\n+          \"description\": \"Name of the user that has access to the Azure SQL server.\"\n         }\n       },\n       \"sqlServerPassword\": {\n         \"type\": \"securestring\",\n         \"metadata\": {\n-          \"description\":  \"Azure SQL server - user password\"\n+          \"description\": \"Password for the user.\"\n         }\n       },\n       \"targetSQLTable\": {\n         \"type\": \"string\",\n         \"metadata\": {\n-          \"description\":  \"Azure SQL table name\"\n-        } \n+          \"description\": \"Table in the Azure SQL Database that will hold the copied data.\"\n+        }\n       }\n     },\n     \"variables\": {\n-      \"apiVersion\": \"2015-10-01\",\n-      \"storageApiVersion\" :  \"2015-05-01-preview\",\n       \"azureSqlLinkedServiceName\": \"AzureSqlLinkedService\",\n       \"azureStorageLinkedServiceName\": \"AzureStorageLinkedService\",\n       \"blobInputDatasetName\": \"BlobInputDataset\",\n       \"sqlOutputDatasetName\": \"SQLOutputDataset\",\n-      \"pipelineName\": \"Blob2SQLPipeline\"    \n+      \"pipelineName\": \"Blob2SQLPipeline\",\n+      \"dataFactoryID\": \"[resourceId('Microsoft.DataFactory/dataFactories/', parameters('dataFactoryName'))]\",","path":"101-data-factory-blob-to-sql-copy/azuredeploy.json","position":null,"original_position":87,"commit_id":"a7fc5c3390df9d7056e794a963d9101701f88e74","original_commit_id":"a7fc5c3390df9d7056e794a963d9101701f88e74","user":{"login":"bmoore-msft","id":10090014,"avatar_url":"https://avatars.githubusercontent.com/u/10090014?v=3","gravatar_id":"","url":"https://api.github.com/users/bmoore-msft","html_url":"https://github.com/bmoore-msft","followers_url":"https://api.github.com/users/bmoore-msft/followers","following_url":"https://api.github.com/users/bmoore-msft/following{/other_user}","gists_url":"https://api.github.com/users/bmoore-msft/gists{/gist_id}","starred_url":"https://api.github.com/users/bmoore-msft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bmoore-msft/subscriptions","organizations_url":"https://api.github.com/users/bmoore-msft/orgs","repos_url":"https://api.github.com/users/bmoore-msft/repos","events_url":"https://api.github.com/users/bmoore-msft/events{/privacy}","received_events_url":"https://api.github.com/users/bmoore-msft/received_events","type":"User","site_admin":false},"body":"One simplification you can make here... if these are only used for dependsOn (looks like they are) then you can simply use the name of the resource since it's defined in the template. e.g. \r\n\r\n\"dependsOn\": [ parameters('dataFactoryName') ]\r\n\r\nThe full resourceId is only needed if it's not defined in this template (e.g. a nested template) or you have resources of different types with the same name...","created_at":"2016-10-10T20:30:43Z","updated_at":"2016-10-10T20:30:47Z","html_url":"https://github.com/Azure/azure-quickstart-templates/pull/2601#discussion_r82677176","pull_request_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/pulls/2601","_links":{"self":{"href":"https://api.github.com/repos/Azure/azure-quickstart-templates/pulls/comments/82677176"},"html":{"href":"https://github.com/Azure/azure-quickstart-templates/pull/2601#discussion_r82677176"},"pull_request":{"href":"https://api.github.com/repos/Azure/azure-quickstart-templates/pulls/2601"}}},"pull_request":{"url":"https://api.github.com/repos/Azure/azure-quickstart-templates/pulls/2601","id":88499315,"html_url":"https://github.com/Azure/azure-quickstart-templates/pull/2601","diff_url":"https://github.com/Azure/azure-quickstart-templates/pull/2601.diff","patch_url":"https://github.com/Azure/azure-quickstart-templates/pull/2601.patch","issue_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/issues/2601","number":2601,"state":"open","locked":false,"title":"blob-sql salesforce-blob sample templates","user":{"login":"spelluru","id":6722422,"avatar_url":"https://avatars.githubusercontent.com/u/6722422?v=3","gravatar_id":"","url":"https://api.github.com/users/spelluru","html_url":"https://github.com/spelluru","followers_url":"https://api.github.com/users/spelluru/followers","following_url":"https://api.github.com/users/spelluru/following{/other_user}","gists_url":"https://api.github.com/users/spelluru/gists{/gist_id}","starred_url":"https://api.github.com/users/spelluru/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/spelluru/subscriptions","organizations_url":"https://api.github.com/users/spelluru/orgs","repos_url":"https://api.github.com/users/spelluru/repos","events_url":"https://api.github.com/users/spelluru/events{/privacy}","received_events_url":"https://api.github.com/users/spelluru/received_events","type":"User","site_admin":false},"body":"### Description of the change\r\n\r\n1. Copy from Azure Blob to Azure SQL template. \r\n2. Copy from Salesforce to Azure Blob\r\n\r\n","created_at":"2016-10-07T20:02:53Z","updated_at":"2016-10-10T20:30:47Z","closed_at":null,"merged_at":null,"merge_commit_sha":"18fb809a01064466d6329c2b3a6063e4a323ae05","assignee":{"login":"bmoore-msft","id":10090014,"avatar_url":"https://avatars.githubusercontent.com/u/10090014?v=3","gravatar_id":"","url":"https://api.github.com/users/bmoore-msft","html_url":"https://github.com/bmoore-msft","followers_url":"https://api.github.com/users/bmoore-msft/followers","following_url":"https://api.github.com/users/bmoore-msft/following{/other_user}","gists_url":"https://api.github.com/users/bmoore-msft/gists{/gist_id}","starred_url":"https://api.github.com/users/bmoore-msft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bmoore-msft/subscriptions","organizations_url":"https://api.github.com/users/bmoore-msft/orgs","repos_url":"https://api.github.com/users/bmoore-msft/repos","events_url":"https://api.github.com/users/bmoore-msft/events{/privacy}","received_events_url":"https://api.github.com/users/bmoore-msft/received_events","type":"User","site_admin":false},"assignees":[{"login":"bmoore-msft","id":10090014,"avatar_url":"https://avatars.githubusercontent.com/u/10090014?v=3","gravatar_id":"","url":"https://api.github.com/users/bmoore-msft","html_url":"https://github.com/bmoore-msft","followers_url":"https://api.github.com/users/bmoore-msft/followers","following_url":"https://api.github.com/users/bmoore-msft/following{/other_user}","gists_url":"https://api.github.com/users/bmoore-msft/gists{/gist_id}","starred_url":"https://api.github.com/users/bmoore-msft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bmoore-msft/subscriptions","organizations_url":"https://api.github.com/users/bmoore-msft/orgs","repos_url":"https://api.github.com/users/bmoore-msft/repos","events_url":"https://api.github.com/users/bmoore-msft/events{/privacy}","received_events_url":"https://api.github.com/users/bmoore-msft/received_events","type":"User","site_admin":false}],"milestone":null,"commits_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/pulls/2601/commits","review_comments_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/pulls/2601/comments","review_comment_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/issues/2601/comments","statuses_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/statuses/a7fc5c3390df9d7056e794a963d9101701f88e74","head":{"label":"spelluru:ADFTemplates","ref":"ADFTemplates","sha":"a7fc5c3390df9d7056e794a963d9101701f88e74","user":{"login":"spelluru","id":6722422,"avatar_url":"https://avatars.githubusercontent.com/u/6722422?v=3","gravatar_id":"","url":"https://api.github.com/users/spelluru","html_url":"https://github.com/spelluru","followers_url":"https://api.github.com/users/spelluru/followers","following_url":"https://api.github.com/users/spelluru/following{/other_user}","gists_url":"https://api.github.com/users/spelluru/gists{/gist_id}","starred_url":"https://api.github.com/users/spelluru/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/spelluru/subscriptions","organizations_url":"https://api.github.com/users/spelluru/orgs","repos_url":"https://api.github.com/users/spelluru/repos","events_url":"https://api.github.com/users/spelluru/events{/privacy}","received_events_url":"https://api.github.com/users/spelluru/received_events","type":"User","site_admin":false},"repo":{"id":70193929,"name":"azure-quickstart-templates","full_name":"spelluru/azure-quickstart-templates","owner":{"login":"spelluru","id":6722422,"avatar_url":"https://avatars.githubusercontent.com/u/6722422?v=3","gravatar_id":"","url":"https://api.github.com/users/spelluru","html_url":"https://github.com/spelluru","followers_url":"https://api.github.com/users/spelluru/followers","following_url":"https://api.github.com/users/spelluru/following{/other_user}","gists_url":"https://api.github.com/users/spelluru/gists{/gist_id}","starred_url":"https://api.github.com/users/spelluru/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/spelluru/subscriptions","organizations_url":"https://api.github.com/users/spelluru/orgs","repos_url":"https://api.github.com/users/spelluru/repos","events_url":"https://api.github.com/users/spelluru/events{/privacy}","received_events_url":"https://api.github.com/users/spelluru/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/spelluru/azure-quickstart-templates","description":"Azure Quickstart Templates","fork":true,"url":"https://api.github.com/repos/spelluru/azure-quickstart-templates","forks_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/forks","keys_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/keys{/key_id}","collaborators_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/teams","hooks_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/hooks","issue_events_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/issues/events{/number}","events_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/events","assignees_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/assignees{/user}","branches_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/branches{/branch}","tags_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/tags","blobs_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/git/refs{/sha}","trees_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/git/trees{/sha}","statuses_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/statuses/{sha}","languages_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/languages","stargazers_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/stargazers","contributors_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/contributors","subscribers_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/subscribers","subscription_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/subscription","commits_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/commits{/sha}","git_commits_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/git/commits{/sha}","comments_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/comments{/number}","issue_comment_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/issues/comments{/number}","contents_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/contents/{+path}","compare_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/compare/{base}...{head}","merges_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/merges","archive_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/downloads","issues_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/issues{/number}","pulls_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/pulls{/number}","milestones_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/milestones{/number}","notifications_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/labels{/name}","releases_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/releases{/id}","deployments_url":"https://api.github.com/repos/spelluru/azure-quickstart-templates/deployments","created_at":"2016-10-06T21:17:50Z","updated_at":"2016-10-06T21:17:56Z","pushed_at":"2016-10-10T17:34:41Z","git_url":"git://github.com/spelluru/azure-quickstart-templates.git","ssh_url":"git@github.com:spelluru/azure-quickstart-templates.git","clone_url":"https://github.com/spelluru/azure-quickstart-templates.git","svn_url":"https://github.com/spelluru/azure-quickstart-templates","homepage":"https://azure.microsoft.com/en-us/documentation/templates","size":71071,"stargazers_count":0,"watchers_count":0,"language":"Shell","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":"master"}},"base":{"label":"Azure:master","ref":"master","sha":"23609921efd73e25a7722e30d03dfbb3e15764c9","user":{"login":"Azure","id":6844498,"avatar_url":"https://avatars.githubusercontent.com/u/6844498?v=3","gravatar_id":"","url":"https://api.github.com/users/Azure","html_url":"https://github.com/Azure","followers_url":"https://api.github.com/users/Azure/followers","following_url":"https://api.github.com/users/Azure/following{/other_user}","gists_url":"https://api.github.com/users/Azure/gists{/gist_id}","starred_url":"https://api.github.com/users/Azure/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Azure/subscriptions","organizations_url":"https://api.github.com/users/Azure/orgs","repos_url":"https://api.github.com/users/Azure/repos","events_url":"https://api.github.com/users/Azure/events{/privacy}","received_events_url":"https://api.github.com/users/Azure/received_events","type":"Organization","site_admin":false},"repo":{"id":34407651,"name":"azure-quickstart-templates","full_name":"Azure/azure-quickstart-templates","owner":{"login":"Azure","id":6844498,"avatar_url":"https://avatars.githubusercontent.com/u/6844498?v=3","gravatar_id":"","url":"https://api.github.com/users/Azure","html_url":"https://github.com/Azure","followers_url":"https://api.github.com/users/Azure/followers","following_url":"https://api.github.com/users/Azure/following{/other_user}","gists_url":"https://api.github.com/users/Azure/gists{/gist_id}","starred_url":"https://api.github.com/users/Azure/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Azure/subscriptions","organizations_url":"https://api.github.com/users/Azure/orgs","repos_url":"https://api.github.com/users/Azure/repos","events_url":"https://api.github.com/users/Azure/events{/privacy}","received_events_url":"https://api.github.com/users/Azure/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Azure/azure-quickstart-templates","description":"Azure Quickstart Templates","fork":false,"url":"https://api.github.com/repos/Azure/azure-quickstart-templates","forks_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/forks","keys_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/teams","hooks_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/hooks","issue_events_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/issues/events{/number}","events_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/events","assignees_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/assignees{/user}","branches_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/branches{/branch}","tags_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/tags","blobs_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/git/refs{/sha}","trees_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/statuses/{sha}","languages_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/languages","stargazers_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/stargazers","contributors_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/contributors","subscribers_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/subscribers","subscription_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/subscription","commits_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/commits{/sha}","git_commits_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/git/commits{/sha}","comments_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/comments{/number}","issue_comment_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/issues/comments{/number}","contents_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/contents/{+path}","compare_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/merges","archive_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/downloads","issues_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/issues{/number}","pulls_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/pulls{/number}","milestones_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/milestones{/number}","notifications_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/labels{/name}","releases_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/releases{/id}","deployments_url":"https://api.github.com/repos/Azure/azure-quickstart-templates/deployments","created_at":"2015-04-22T18:19:03Z","updated_at":"2016-10-10T13:24:42Z","pushed_at":"2016-10-10T19:34:25Z","git_url":"git://github.com/Azure/azure-quickstart-templates.git","ssh_url":"git@github.com:Azure/azure-quickstart-templates.git","clone_url":"https://github.com/Azure/azure-quickstart-templates.git","svn_url":"https://github.com/Azure/azure-quickstart-templates","homepage":"https://azure.microsoft.com/en-us/documentation/templates","size":91125,"stargazers_count":1433,"watchers_count":1433,"language":"Shell","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":2057,"mirror_url":null,"open_issues_count":232,"forks":2057,"open_issues":232,"watchers":1433,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Azure/azure-quickstart-templates/pulls/2601"},"html":{"href":"https://github.com/Azure/azure-quickstart-templates/pull/2601"},"issue":{"href":"https://api.github.com/repos/Azure/azure-quickstart-templates/issues/2601"},"comments":{"href":"https://api.github.com/repos/Azure/azure-quickstart-templates/issues/2601/comments"},"review_comments":{"href":"https://api.github.com/repos/Azure/azure-quickstart-templates/pulls/2601/comments"},"review_comment":{"href":"https://api.github.com/repos/Azure/azure-quickstart-templates/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Azure/azure-quickstart-templates/pulls/2601/commits"},"statuses":{"href":"https://api.github.com/repos/Azure/azure-quickstart-templates/statuses/a7fc5c3390df9d7056e794a963d9101701f88e74"}}}}
{"actor":{"display_login":"bmoore-msft"}}