https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/AFFINITY-2 affinity <p>Please convert all usages of System.out.println to proper log statements:</p> <p>ubdir affinity/<br/> Running command: grep -rn System.out .<br/> ./affinity/api/src/main/java/org/opendaylight/affinity/affinity/AffinityGroup.java:98: System.out.println("Printing affinity group " + this.name);<br/> ./affinity/api/src/main/java/org/opendaylight/affinity/affinity/AffinityIdentifier.java:26: System.out.println(name);<br/> ./affinity/api/src/main/java/org/opendaylight/affinity/affinity/AffinityLink.java:84: System.out.println("Printing affinity attribute: " + attr.type);<br/> ./affinity/api/src/test/java/org/opendaylight/affinity/affinity/AffinityGroupTest.java:35: System.out.println(ret3);<br/> ./affinity/api/src/test/java/org/opendaylight/affinity/affinity/AffinityGroupTest.java:58: System.out.println("Affinity group size is " + ag1.size());<br/> ./affinity/implementation/src/test/java/org/opendaylight/affinity/affinity/internal/AffinityManagerImplTest.java:50: System.out.println(ret3);<br/> ./affinity/implementation/src/test/java/org/opendaylight/affinity/affinity/internal/AffinityManagerImplTest.java:72: System.out.println("Affinity group size is " + ag1.size());<br/> ./affinity/implementation/src/test/java/org/opendaylight/affinity/affinity/internal/AffinityManagerImplTest.java:111: System.out.println("affinity link " + al3.getName());<br/> ./affinity/implementation/src/test/java/org/opendaylight/affinity/affinity/internal/AffinityManagerImplTest.java:115: System.out.println("flow with from=" + flow.getKey() + " to=" + flow.getValue());<br/> ./affinity/implementation/src/test/java/org/opendaylight/affinity/affinity/internal/AffinityManagerImplTest.java:121: // System.out.println("Affinity group (as Hosts) = " + ag1.getName());<br/> ./affinity/implementation/src/test/java/org/opendaylight/affinity/affinity/internal/AffinityManagerImplTest.java:125: // System.out.println("host = " + h.getNetworkAddressAsString());<br/> ./affinity/implementation/src/test/java/org/opendaylight/affinity/affinity/internal/AffinityManagerImplTest.java:129: System.out.println("Affinity group (as Affinity Identifiers) = " + ag1.getName());<br/> ./affinity/implementation/src/test/java/org/opendaylight/affinity/affinity/internal/AffinityManagerImplTest.java:137: System.out.println("Affinity link = " + al1.getName());<br/> ./affinity/implementation/src/test/java/org/opendaylight/affinity/affinity/internal/AffinityManagerImplTest.java:141: System.out.println("flow " + "from: " + flow.getKey().toString() + "to: " + flow.getValue().toString());<br/> ./affinity/integrationtest/src/test/java/org/opendaylight/affinity/affinity/internal/AffinityManagerIT.java:112: System.out.println("&gt;&gt;&gt; " + b<span class="error">&#91;i&#93;</span>.getSymbolicName() + " " + stateToString(state));</p> <p>Operating System: Mac OS<br/> Platform: PC</p> AFFINITY-2

Please convert System.out.println usage to proper log statements

Improvement Resolved Done Unassigned Ed Warnicke Wed, 15 Jan 2014 00:59:36 +0000 Thu, 19 Oct 2017 21:19:09 +0000 Mon, 20 Jan 2014 19:22:59 +0000 unspecified General 0 1 <p>I've checked in a change that uses logger for OSGi modules. <br/> There are a few println statements in unit test code, which seems ok to me. <br/> If this is not acceptable, I'll take care of the test code in a second commit. <br/> Marking this as fixed. Please re-open if test code must be converted.</p> Development External issue ID 296 External issue URL Rank 0|i02u3b: