Az.DataFactory update to include Global Parameters

[Ceespino]

Description

Added global parameters to Data Factory

Checklist

• I have read the Submitting Changes section of CONTRIBUTING.md
• The title of the PR is clear and informative
• The appropriate ChangeLog.md file(s) has been updated:
  • For any service, the ChangeLog.md file can be found at src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
  • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header -- no new version header should be added
• The PR does not introduce breaking changes
• If applicable, the changes made in the PR have proper test coverage
• [] For public API changes to cmdlets:
  • a cmdlet design review was approved for the changes in this repository (Microsoft internal only)
  • the markdown help files have been regenerated using the commands listed here

[adxsdkps]

Can one of the admins verify this patch?

[VeryEarly]

• Can you help me understand where this "global parameters" comes from?

• If it's needed, it should be called "GlobalParameter"

• Please update and check in Set-AzDataFactoryV2.md only

[Ceespino]

• Can you help me understand where this "global parameters" comes from?
• If it's needed, it should be called "GlobalParameter"
• Please update and check in Set-AzDataFactoryV2.md only

Global Parameters is a new property in the Data Factory entity. It is basically a dictionary that contains global constants that can be referenced and used by its subresources.

I think it should be called "Global Parameters" since that's the name that it is refered to in the SDK, factory entity and in our UI where our customers can author and use them.

Ran the tool again to only update the Get and Set AzDataFactoryV2 cmdlet since those are the ones affected.

[Ceespino]

@shawnxzq to review the changes and visibility.

[VeryEarly]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[VeryEarly]

• Can you help me understand where this "global parameters" comes from?
• If it's needed, it should be called "GlobalParameter"
• Please update and check in Set-AzDataFactoryV2.md only

Global Parameters is a new property in the Data Factory entity. It is basically a dictionary that contains global constants that can be referenced and used by its subresources.

I think it should be called "Global Parameters" since that's the name that it is refered to in the SDK, factory entity and in our UI where our customers can author and use them.

Ran the tool again to only update the Get and Set AzDataFactoryV2 cmdlet since those are the ones affected.

Please follow powershell convention and use singularity for parameter names
https://github.com/Azure/azure-powershell/blob/master/documentation/development-docs/design-guidelines/parameter-best-practices.md

[Ceespino]

• Can you help me understand where this "global parameters" comes from?
• If it's needed, it should be called "GlobalParameter"
• Please update and check in Set-AzDataFactoryV2.md only

Global Parameters is a new property in the Data Factory entity. It is basically a dictionary that contains global constants that can be referenced and used by its subresources.
I think it should be called "Global Parameters" since that's the name that it is refered to in the SDK, factory entity and in our UI where our customers can author and use them.
Ran the tool again to only update the Get and Set AzDataFactoryV2 cmdlet since those are the ones affected.

Please follow powershell convention and use singularity for parameter names
https://github.com/Azure/azure-powershell/blob/master/documentation/development-docs/design-guidelines/parameter-best-practices.md

Updated name to GlobalParameterDefinition

[Ceespino]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 12290 in repo Azure/azure-powershell

[Ceespino]

@VeryEarly Can we include this in the next release please? (7/14)

[VeryEarly]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[VeryEarly]

yes we can, merged.

[shawnxzq]

Looks good to me, thanks @Ceespino