-
Notifications
You must be signed in to change notification settings - Fork 1k
[Server] ActivateSessionAsync / CreateSessionAsync / FindServersAsync / GetEndpointsAsync / CloseSessionAsync #3225
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft
romanett
wants to merge
38
commits into
OPCFoundation:master
Choose a base branch
from
romanett:feat/moreAsync
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 37 commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
9c7d1b3
Implement management Methods from INodeManager & INodeManager2 into I…
romanett f8aff1f
Use Async methods in MasterNodeManager
romanett 5359998
Update ReferenceServerWithLimits
romanett 6a32d33
fix test
romanett 5199ea9
fix typo
romanett d125051
propagate MonitoredItem Id using Func<uint> getNextMonitoredItemId in…
romanett 3ca5082
update SampleNodeManager
romanett 58aa6a9
Update MemoryBufferNodeManager
romanett 34bc4b2
Merge branch 'feat/AsyncReadyCreateMonitoredItems' into feat/AsyncNod…
romanett 4432e45
Implement Async MI Management methods in MasterNodeManager
romanett 6b8a32b
fix indentation
romanett 006d3cb
Merge remote-tracking branch 'origin/master' into feat/asyncNodeManager
romanett 6bfc37f
remove obsolete sync code
romanett f12b7d5
use Class MonitoredItem Id to propagate MonitoredItemIds
romanett 0e4ec62
use lock
romanett 0d7de2d
rename, make instance readonly
romanett ce782b6
rename to MonitoredItemIdFactory
romanett 4ccb6a7
Add Concurrency test
romanett b24c278
merge
romanett d2d0db9
fix build
romanett 6a3476c
expose sync nodeManager
romanett d077ebb
Merge branch 'master' into feat/asyncNodeManager
marcschier 9ec1d43
merge
romanett 22384c2
fix build, fix analyzer messages
romanett 1822eb4
Allow to register IAsyncNodeMangers, Add SyncNodeManager Adapter.
romanett 5f29b72
Implement adapter
romanett 13556cd
Add RemoveNamespaceMananager(IASyncNodeManager)
romanett c33dfd1
Allow AsyncNodeManagers in ReferenceServer
romanett 86c9d23
merge
romanett 6ed7aae
Add documentation
romanett acf25a5
Create real sync GetManagerHandle implementation
romanett c9be3ea
merge
romanett f00199f
fix typo
romanett 8618df4
fix adapter, fix typos
romanett a1c6c47
document async method call
romanett 7383079
Merge remote-tracking branch 'origin/master' into feat/moreAsync
romanett ec721ad
ActivateSessionAsync CreateSessionAsync
romanett 99c7050
Add StartAsync StopAsync to ServerBase
romanett File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should have an abstraction here when we expose the lock, something like ILock, then we have the opportunity to change this later, and dont repeat what we have already run into with the ubiqu SyncLock, SyncObject etc. What do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you expose like an AsyncLock or just a thin wrapper around semaphoreslim?