Skip to content

[DataFactory]Powershell Release#29024

Merged
wyunchi-ms merged 2 commits intoAzure:mainfrom
Jingshu918:main
Jan 5, 2026
Merged

[DataFactory]Powershell Release#29024
wyunchi-ms merged 2 commits intoAzure:mainfrom
Jingshu918:main

Conversation

@Jingshu918
Copy link
Copy Markdown
Contributor

@Jingshu918 Jingshu918 commented Jan 5, 2026

Description

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

Copilot AI review requested due to automatic review settings January 5, 2026 09:26
@azure-client-tools-bot-prd
Copy link
Copy Markdown

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds support for enhanced connection properties for JiraV2 and NetezzaV2 in the Azure Data Factory PowerShell module. The changes include SDK model updates with new properties and a ChangeLog entry documenting the enhancement.

  • Added new connection properties for Netezza linked services (server, port, uid, database, securityLevel)
  • Added new dataset properties for Jira V2 (schema and table properties)
  • Updated SDK commit reference to regenerate models from latest Azure REST API specifications

Reviewed changes

Copilot reviewed 2 out of 7 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
src/DataFactory/DataFactory/ChangeLog.md Adds release note for JiraV2 and NetezzaV2 connection properties support
src/DataFactory/DataFactory.Management.Sdk/README.md Updates commit hash for SDK regeneration
src/DataFactory/DataFactory.Management.Sdk/Generated/Models/NetezzaSecurityLevelType.cs Defines new security level constants for Netezza connections
src/DataFactory/DataFactory.Management.Sdk/Generated/Models/NetezzaLinkedServiceTypeProperties.cs Adds new connection properties (server, port, uid, database, securityLevel)
src/DataFactory/DataFactory.Management.Sdk/Generated/Models/NetezzaLinkedService.cs Adds new connection properties to the linked service model
src/DataFactory/DataFactory.Management.Sdk/Generated/Models/JiraTableDatasetTypeProperties.cs Defines new dataset properties for Jira with V2 support
src/DataFactory/DataFactory.Management.Sdk/Generated/Models/JiraObjectDataset.cs Adds schema and table properties for Jira V2 datasets

Comment on lines +106 to +109
/// Gets or sets specifies the security level for the driver connection to the
/// data store. PreferredUnSecured : prefer unsecured, allow fallback to
/// secured connection if required. OnlyUnSecured : strictly unsecured, no
/// fallback. Possible values include: 'PreferredUnSecured', 'OnlyUnSecured'
Copy link

Copilot AI Jan 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spelling error: "PreferredUnSecured" should be "PreferredUnsecured" and "OnlyUnSecured" should be "OnlyUnsecured". The word "unsecured" should not have a capital 'U' in the middle.

Copilot uses AI. Check for mistakes.
Comment on lines +65 to +67
/// PreferredUnSecured : prefer unsecured, allow fallback to secured connection
/// if required. OnlyUnSecured : strictly unsecured, no fallback.
/// Possible values include: &#39;PreferredUnSecured&#39;, &#39;OnlyUnSecured&#39;</param>
Copy link

Copilot AI Jan 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spelling error: "PreferredUnSecured" should be "PreferredUnsecured" and "OnlyUnSecured" should be "OnlyUnsecured". The word "unsecured" should not have a capital 'U' in the middle.

Suggested change
/// PreferredUnSecured : prefer unsecured, allow fallback to secured connection
/// if required. OnlyUnSecured : strictly unsecured, no fallback.
/// Possible values include: &#39;PreferredUnSecured&#39;, &#39;OnlyUnSecured&#39;</param>
/// PreferredUnsecured : prefer unsecured, allow fallback to secured connection
/// if required. OnlyUnsecured : strictly unsecured, no fallback.
/// Possible values include: &#39;PreferredUnsecured&#39;, &#39;OnlyUnsecured&#39;</param>

Copilot uses AI. Check for mistakes.
Comment on lines +129 to +131
/// data store. PreferredUnSecured : prefer unsecured, allow fallback to
/// secured connection if required. OnlyUnSecured : strictly unsecured, no
/// fallback. Possible values include: &#39;PreferredUnSecured&#39;, &#39;OnlyUnSecured&#39;
Copy link

Copilot AI Jan 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spelling error: "PreferredUnSecured" should be "PreferredUnsecured" and "OnlyUnSecured" should be "OnlyUnsecured". The word "unsecured" should not have a capital 'U' in the middle.

Suggested change
/// data store. PreferredUnSecured : prefer unsecured, allow fallback to
/// secured connection if required. OnlyUnSecured : strictly unsecured, no
/// fallback. Possible values include: &#39;PreferredUnSecured&#39;, &#39;OnlyUnSecured&#39;
/// data store. PreferredUnsecured : prefer unsecured, allow fallback to
/// secured connection if required. OnlyUnsecured : strictly unsecured, no
/// fallback. Possible values include: &#39;PreferredUnsecured&#39;, &#39;OnlyUnsecured&#39;

Copilot uses AI. Check for mistakes.
- Additional information about change #1
-->
## Upcoming Release
* Added more support for JiraV2 and NetezzaV2 connection properties.
Copy link

Copilot AI Jan 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The ChangeLog entry should explain the acronyms "JiraV2" and "NetezzaV2" on first use, as the primary audience is Azure PowerShell users who may not be familiar with these terms. Consider expanding to something like: "Added more support for Jira V2 (Jira Version 2) and Netezza V2 (Netezza Version 2) connection properties." Additionally, if this change relates to a GitHub issue, please reference it using the format [#12345].

Copilot generated this review using guidance from repository custom instructions.
Comment on lines +16 to +17
public const string PreferredUnSecured = "PreferredUnSecured";
public const string OnlyUnSecured = "OnlyUnSecured";
Copy link

Copilot AI Jan 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The constant names "PreferredUnSecured" and "OnlyUnSecured" use inconsistent capitalization. The prefix "Un" should be lowercase to follow proper camelCase conventions. Consider renaming to "PreferredUnsecured" and "OnlyUnsecured".

Suggested change
public const string PreferredUnSecured = "PreferredUnSecured";
public const string OnlyUnSecured = "OnlyUnSecured";
public const string PreferredUnsecured = "PreferredUnSecured";
public const string OnlyUnsecured = "OnlyUnSecured";

Copilot uses AI. Check for mistakes.
Comment on lines +43 to +46
/// <param name="securityLevel">Specifies the security level for the driver connection to the data store.
/// PreferredUnSecured : prefer unsecured, allow fallback to secured connection
/// if required. OnlyUnSecured : strictly unsecured, no fallback.
/// Possible values include: &#39;PreferredUnSecured&#39;, &#39;OnlyUnSecured&#39;</param>
Copy link

Copilot AI Jan 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spelling error: "PreferredUnSecured" should be "PreferredUnsecured" (lowercase 'u' in 'unsecured'). The word "unsecured" should not have a capital 'U' in the middle.

Copilot uses AI. Check for mistakes.
@isra-fel
Copy link
Copy Markdown
Member

isra-fel commented Jan 5, 2026

/azp run

@azure-pipelines
Copy link
Copy Markdown
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

@wyunchi-ms wyunchi-ms merged commit 73a4498 into Azure:main Jan 5, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants