[CONTROLLER-246] Eliminate the use of xtend in md-sal/sal-rest-connector Created: 31/Mar/14 Updated: 25/Jul/23 Resolved: 20/May/14 |
|
| Status: | Resolved |
| Project: | controller |
| Component/s: | mdsal |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | ||
| Reporter: | Robert Varga | Assignee: | Tom Pantelis |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| Issue Links: |
|
||||||||
| Description |
|
The single should be trivial to write in Java, reducing both compile-time and run-time overhead. ./opendaylight/md-sal/sal-rest-connector/src/main/java/org/opendaylight/controller/sal/restconf/impl/BrokerFacade.xtend Also be sure to drop the plugin/lib dependency from pom.xml. |
| Comments |
| Comment by Robert Varga [ 31/Mar/14 ] |
|
Applies to opendaylight/md-sal/sal-rest-connector/src/main/java/org/opendaylight/controller/sal/restconf/impl/RestconfImpl.xtend as well. The final file is valid templating, so the dependency cannot be completely dropped yet. |
| Comment by Tom Pantelis [ 08/May/14 ] |
|
I converted the BrokerFacade and ControllerContext classes with https://git.opendaylight.org/gerrit/#/c/6821/ Also added unit tests for BrokerFacade. The code coverage for ControllerContext is already pretty good via existing unit tests. RestConfImpl is being converted by |