Details
-
Improvement
-
Status: Resolved
-
Resolution: Won't Do
-
None
-
None
-
None
-
None
-
Operating System: All
Platform: All
Description
The default InstanceIdentifierBuilder needs to be a copy builder, such that it has stable state and the following code is valid (seen in the wild):
InstanceIdentifierBuilder<Foo> foob = InstanceIdentfier.builder(Foo.class);
dataProviderService.removeOperationalData(foob.augmentation(Bar.class).build));
dataProviderService.removeOperationalData(foob.augmentation(Baz.class).build));
Once it is stable, also add hashCode()/equals() so it can be kept around safely.
The current mutable version may be retained as an explicit MutableInstanceIdentifierBuilder or something of that kind.
Attachments
| # | Subject | Branch | Project | Status | CR | V |
|---|---|---|---|---|---|---|
| 83356,1 | Bump versions for mdsal-5.0.0 | master | mdsal | Status: MERGED | +2 | +1 |
| 83379,2 | Use jdk11 for mdsal master | master | releng/builder | Status: MERGED | +2 | +1 |
| 84924,13 | Bump version odlparent->6.0.0,mdsal->5.0.3 | master | genius | Status: MERGED | +2 | +1 |
| 84931,9 | Bump Version odlparent->6.0.0,mdsal->5.0.3 | master | sfc | Status: MERGED | +2 | +1 |
| 84986,21 | Bump odlparent->6.0.0,mdsal->5.0.3 | master | netvirt | Status: MERGED | +2 | +1 |
| 85549,1 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | controller | Status: MERGED | +2 | +1 |
| 85615,1 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | aaa | Status: MERGED | +2 | +1 |
| 85616,1 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | serviceutils | Status: MERGED | +2 | +1 |
| 85618,1 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | netconf | Status: MERGED | +2 | +1 |
| 85625,1 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | bgpcep | Status: MERGED | +2 | +1 |
| 85627,5 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | openflowplugin | Status: MERGED | +2 | +1 |
| 85630,1 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | daexim | Status: MERGED | +2 | +1 |
| 85631,1 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | coe | Status: MERGED | +2 | +1 |
| 85632,2 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | ovsdb | Status: MERGED | +2 | +1 |
| 85633,1 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | neutron | Status: MERGED | +2 | +1 |
| 85634,2 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | lispflowmapping | Status: MERGED | +2 | +1 |
| 85635,1 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | genius | Status: MERGED | +2 | +1 |
| 85636,2 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | sfc | Status: MERGED | +2 | +1 |
| 85637,4 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | netvirt | Status: MERGED | +2 | +1 |
| 85638,1 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | integration/distribution | Status: MERGED | +2 | +1 |
| 85639,1 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | archetypes | Status: MERGED | +2 | +1 |
| 85841,3 | Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 | master | jsonrpc | Status: MERGED | +2 | +1 |
| 86452,2 | Add mdsal-5.0.x jobs | master | releng/builder | Status: MERGED | +2 | +1 |
| 86583,1 | Bump to odlparent-6.0.3/mdsal-5.0.4 | master | detnet | Status: MERGED | +2 | +1 |
| 86994,2 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | controller | Status: MERGED | +2 | +1 |
| 86995,2 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | aaa | Status: MERGED | +2 | +1 |
| 86996,3 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | netconf | Status: MERGED | +2 | +1 |
| 86997,2 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | serviceutils | Status: MERGED | +2 | +1 |
| 86998,2 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | bgpcep | Status: MERGED | +2 | +1 |
| 86999,3 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | coe | Status: MERGED | +2 | +1 |
| 87000,2 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | ovsdb | Status: MERGED | +2 | +1 |
| 87001,2 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | openflowplugin | Status: MERGED | +2 | +1 |
| 87002,2 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | daexim | Status: MERGED | +2 | +1 |
| 87003,2 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | jsonrpc | Status: MERGED | +2 | +1 |
| 87004,2 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | lispflowmapping | Status: MERGED | +2 | +1 |
| 87005,3 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | neutron | Status: MERGED | +2 | +1 |
| 87006,2 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | genius | Status: MERGED | +2 | +1 |
| 87007,2 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | sfc | Status: MERGED | +2 | +1 |
| 87008,2 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | netvirt | Status: MERGED | +2 | +1 |
| 87009,3 | Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | archetypes | Status: MERGED | +2 | +1 |
| 87010,3 | Bump odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 | master | integration/distribution | Status: MERGED | +2 | +1 |
| 89193,4 | Bump odlparent/yangtools/mdsal | stable/magnesium | controller | Status: MERGED | +2 | +1 |
| 89194,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | aaa | Status: MERGED | +2 | +1 |
| 89196,3 | Bump odlparent/yangtools/mdsal | stable/magnesium | netconf | Status: MERGED | +2 | +1 |
| 89336,3 | Bump odlparent/yangtools/mdsal | stable/magnesium | serviceutils | Status: MERGED | +2 | +1 |
| 89337,3 | Bump odlparent/yangtools/mdsal | stable/magnesium | bgpcep | Status: MERGED | +2 | +1 |
| 89338,2 | Bump odlparent/yangtools/mdsal | stable/magnesium | ovsdb | Status: MERGED | +2 | +1 |
| 89339,2 | Bump odlparent/yangtools/mdsal | stable/magnesium | openflowplugin | Status: MERGED | +2 | +1 |
| 89340,3 | Bump odlparent/yangtools/mdsal | stable/magnesium | daexim | Status: MERGED | +2 | +1 |
| 89341,2 | Bump odlparent/yangtools/mdsal | stable/magnesium | jsonrpc | Status: MERGED | +2 | +1 |
| 89342,2 | Bump odlparent/yangtools/mdsal | stable/magnesium | neutron | Status: MERGED | +2 | +1 |
| 89343,2 | Bump odlparent/yangtools/mdsal | stable/magnesium | genius | Status: MERGED | +2 | +1 |
| 89344,2 | Bump odlparent/yangtools/mdsal | stable/magnesium | lispflowmapping | Status: MERGED | +2 | +1 |
| 89345,2 | Bump odlparent/yangtools/mdsal | stable/magnesium | netvirt | Status: MERGED | +2 | +1 |
| 89346,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | integration/distribution | Status: MERGED | +2 | +1 |
| 91045,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | serviceutils | Status: MERGED | +2 | +1 |
| 91046,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | controller | Status: MERGED | +2 | +1 |
| 91047,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | aaa | Status: MERGED | +2 | +1 |
| 91048,3 | Bump odlparent/yangtools/mdsal | stable/magnesium | bgpcep | Status: MERGED | +2 | +1 |
| 91049,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | ovsdb | Status: MERGED | +2 | +1 |
| 91050,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | openflowplugin | Status: MERGED | +2 | +1 |
| 91051,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | daexim | Status: MERGED | +2 | +1 |
| 91052,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | neutron | Status: MERGED | +2 | +1 |
| 91053,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | genius | Status: MERGED | +2 | +1 |
| 91074,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | lispflowmapping | Status: MERGED | +2 | +1 |
| 91075,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | netvirt | Status: MERGED | +2 | +1 |
| 91080,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | integration/distribution | Status: MERGED | +2 | +1 |
| 91083,2 | Bump odlparent/yangtools/mdsal | stable/magnesium | netconf | Status: MERGED | +2 | +1 |
| 91088,1 | Bump odlparent/yangtools/mdsal | stable/magnesium | jsonrpc | Status: MERGED | +2 | +1 |
| 91394,2 | Bump odlparent/yangtools/mdsal | stable/magnesium | integration/distribution | Status: MERGED | +2 | +1 |
| 93795,2 | Bump upstream versions | stable/magnesium | controller | Status: MERGED | +2 | +1 |
| 93796,2 | Bump upstream versions | stable/magnesium | aaa | Status: MERGED | +2 | +1 |
| 93797,2 | Bump upstream versions | stable/magnesium | serviceutils | Status: MERGED | +2 | +1 |
| 93798,2 | Bump upstream versions | stable/magnesium | netconf | Status: MERGED | +2 | +1 |
| 93799,2 | Bump upstream versions | stable/magnesium | ovsdb | Status: MERGED | +2 | +1 |
| 93800,2 | Bump upstream versions | stable/magnesium | openflowplugin | Status: MERGED | +2 | +1 |
| 93801,2 | Bump upstream versions | stable/magnesium | daexim | Status: MERGED | +2 | +1 |
| 93802,2 | Bump upstream versions | stable/magnesium | jsonrpc | Status: MERGED | +2 | +1 |
| 93803,2 | Bump upstream versions | stable/magnesium | genius | Status: MERGED | +2 | +1 |
| 93804,2 | Bump upstream versions | stable/magnesium | lispflowmapping | Status: MERGED | +2 | +1 |
| 93805,2 | Bump upstream versions | stable/magnesium | neutron | Status: MERGED | +2 | +1 |
| 93806,2 | Bump upstream versions | stable/magnesium | bgpcep | Status: MERGED | +2 | +1 |
| 93807,2 | Bump upstream versions | stable/magnesium | netvirt | Status: MERGED | +2 | +1 |
| 93814,2 | Bump upstream versions | stable/magnesium | integration/distribution | Status: MERGED | +2 | +1 |
| 95804,1 | Remove mdsal-5.0.x jobs | master | releng/builder | Status: MERGED | +2 | +1 |