[CONTROLLER-112] Please convert System.out.println usage to proper log statements Created: 15/Jan/14 Updated: 25/Jul/23 Resolved: 05/May/15 |
|
| Status: | Resolved |
| Project: | controller |
| Component/s: | adsal |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | ||
| Reporter: | Ed Warnicke | Assignee: | Unassigned |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: Mac OS |
||
| Issue Links: |
|
||||||||
| Description |
|
Please convert all usages of System.out.println to proper log statements: Running command: grep -rn System.out . |
| Comments |
| Comment by Sahil Shah [ 18/Jan/14 ] |
|
Hi Ed, -Sahil |
| Comment by Giovanni Meo [ 23/Jan/14 ] |
|
For the following files: are used in CLI hence the output has to be given back to user interactively. The follow file in only used in integration tests so output has to be given interactively: The follow file are only used in demo code hence output has to be given interactively: |
| Comment by Jyotsna Priya [ 23/Jan/14 ] |
|
Hi Giovanni, Apart from the files mentioned by you, the rest of the files can be divided into three categories: *Impl.java, *.xtend and *Test.java *Test.java files are: Again these files are used in tests so output has to be given interactively. So changes are not required. *.xtend file is: and *Impl.java files are: Please suggest if changes in *Impl.java and *.xtend files are required or not. -Jyotsna |
| Comment by Giovanni Meo [ 23/Jan/14 ] |
|
Hi Jyotsna, the files: *.xtend file is: and *Impl.java files are: are required to be changed, these are used at runtime. Thanks, |
| Comment by Jyotsna Priya [ 28/Jan/14 ] |
|
As per the comments on https://git.opendaylight.org/gerrit/#/c/4675/ , none of the files require any change. So this bug should be closed. |
| Comment by Tony Tkacik [ 13/Nov/14 ] |
|
Added as blocking issue for Topic: Decrease Technical Debt. |
| Comment by Carol Sanders [ 04/May/15 ] |
|
This bug is part of the project to Move all ADSAL associated component bugs to ADSAL |