Skip to content

Review KNet Connect SDK configuration (#1429)#1430

Merged
masesdevelopers merged 1 commit intorelease/2.9.Xfrom
1428-the-configure-method-in-net-does-not-receive-the-default-values-2-9-x
Mar 12, 2026
Merged

Review KNet Connect SDK configuration (#1429)#1430
masesdevelopers merged 1 commit intorelease/2.9.Xfrom
1428-the-configure-method-in-net-does-not-receive-the-default-values-2-9-x

Conversation

@masesdevelopers
Copy link
Contributor

@masesdevelopers masesdevelopers commented Mar 12, 2026

Description

  • Review KNet Connect SDK configuration
    The configuration is reviewed to identify and propagate default values defined in JVM side. Adds a new IKNetCommonConfiguration to simplify the access to configuration properties and updates the signature of the methods to introduce the new behavior. Task configuration in Source/Sink connector wasn't changed since it is a pass-through between connector and tasks

  • Updates KNetTask Start method adding support for string only configuration values

  • Adds IKNetTaskConfiguration to support configuration management from tasks

  • Potential fix for pull request finding 'Missed 'readonly' opportunity'

  • Fix task data exchange

  • Fix NullReferenceException and adds logs when methods fails reporting .NET info

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

* Review KNet Connect SDK configuration

The configuration is reviewed to identify and propagate default values defined in JVM side. Adds a new IKNetCommonConfiguration to simplify the access to configuration properties and updates the signature of the methods to introduce the new behavior. Task configuration in Source/Sink connector wasn't changed since it is a pass-through between connector and tasks

* Updates KNetTask Start method adding support for string only configuration values

* Adds IKNetTaskConfiguration to support configuration management from tasks

* Potential fix for pull request finding 'Missed 'readonly' opportunity'

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>

* Fix task data exchange

* Fix NullReferenceException and adds logs when methods fails reporting .NET info

---------

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
@masesdevelopers masesdevelopers self-assigned this Mar 12, 2026
@masesdevelopers masesdevelopers added bug Something isn't working enhancement New feature or request KNetConnectSDK KNetConnectSDK related issue java Pull requests that update Java code .NET Pull requests that update .net code labels Mar 12, 2026
@masesdevelopers masesdevelopers merged commit 7e79d63 into release/2.9.X Mar 12, 2026
168 of 170 checks passed
@masesdevelopers masesdevelopers deleted the 1428-the-configure-method-in-net-does-not-receive-the-default-values-2-9-x branch March 12, 2026 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request java Pull requests that update Java code KNetConnectSDK KNetConnectSDK related issue .NET Pull requests that update .net code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant