[
http://mifosforge.jira.com/browse/MIFOS-3923?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Lukasz Chudy updated MIFOS-3923:
--------------------------------
Comment: was deleted
(was: Removed labels: maven techdebt)
> Remove all System.out (replace by CORRECT logging) - and ensure they don't
> come back
> ------------------------------------------------------------------------------------
>
> Key: MIFOS-3923
> URL: http://mifosforge.jira.com/browse/MIFOS-3923
> Project: mifos
> Issue Type: Bug
> Components: Build and Testing
> Reporter: Michael Vorburger
> Assignee: mifosdeveloperqueue
> Labels: Testing, maven, techdebt
> Fix For: Unscheduled
>
>
> I've found 69x System.out, which I'd argue are bugs and should be removed and
> replaced by CORRECT logging. System.out is a performance bottleneck (as it,
> usually, synchronizes on the console), not relevant in single user tests &
> dev, but measurable in production / load scenarios.
> In tests they are less of a problem, in src/main they should be absolutely
> forbidden.
> There are also 12 System.err - I had a quick look through them and they are
> all in test or helpers (in src/main) which are probably called from tools
> main() and not running Web app code, so may be fine?
> Ideally this issue should include some action to ensure they don't come
> back... e.g. make sure your shared FindBugs or Checkstyle config (if you do
> this?) are configured to flag it up red - or just fail the build if any are
> re-introduced, that usually helps ;).
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
http://mifosforge.jira.com/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure
contains a definitive record of customers, application performance,
security threats, fraudulent activity, and more. Splunk takes this
data and makes sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-novd2d
_______________________________________________
Mifos-issues mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mifos-issues
