Code Review / netconf.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: d350ffc) | patch
Refactor SubscriptionHolder 91/116491/8
authorRobert Varga <robert.varga@pantheon.tech>
Thu, 8 May 2025 01:32:31 +0000 (03:32 +0200)
committerRobert Varga <nite@hq.sk>
Thu, 8 May 2025 17:08:57 +0000 (17:08 +0000)
commit424e1a7fbe0c3cda0b016a9a579d5b7f7cab747f
treefa35b96412c3c7b1e872b781d38004a4b13125a1tree | snapshot
parentd350ffcd6cbad06b4db49b9410646e982f9af6e4commit | diff
Refactor SubscriptionHolder

Move SubscriptionHolder to being an internal detail of
AbstractRestconfStreamRegistry, allowing us to fuse subscription
creation with registration to control session.

This eliminates the need for SubscriptionState.STARTED, as we can
just create the subscription as ACTIVE instead.

JIRA: NETCONF-1449
Change-Id: I25698ee05ec2586776308a1b48f1846d066205cf
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
apps/restconf-subscription/src/main/java/org/opendaylight/restconf/subscription/EstablishSubscriptionRpc.java diff | blob | history
apps/restconf-subscription/src/main/java/org/opendaylight/restconf/subscription/SubscriptionHolder.java [deleted file] blob | history
apps/restconf-subscription/src/test/java/org/opendaylight/restconf/subscription/EstablishSubscriptionRpcTest.java diff | blob | history
protocol/restconf-server-spi/src/main/java/org/opendaylight/restconf/server/spi/AbstractRestconfStreamRegistry.java diff | blob | history
protocol/restconf-server-spi/src/main/java/org/opendaylight/restconf/server/spi/RestconfStream.java diff | blob | history
protocol/restconf-server-spi/src/test/java/org/opendaylight/restconf/server/spi/SubscriptionStateTest.java diff | blob | history
Netconf