Bug 427720 - JobManager writes message "Job found still running..." to standard error.
Summary: JobManager writes message "Job found still running..." to standard error.
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Runtime (show other bugs)
Version: 3.8.2   Edit
Hardware: PC All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: platform-runtime-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-08 01:39 EST by William Chen CLA
Modified: 2014-02-10 04:21 EST (History)
0 users

See Also:


Attachments
An example of unwanted "Job found still running..." (22.09 KB, image/png)
2014-02-08 01:47 EST, William Chen CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description William Chen CLA 2014-02-08 01:39:17 EST
When eclipse platform is being shutdown, JobManager checks if there are running jobs which are not canceled by plugins that schedule them. If such jobs are found, it report error message like:

"Job found still running after platform shutdown..."

In addition to logging the message, it also writes the message to the standard error, which are unwanted in some case. Eclipse applications without GUI, for example a command line tool, may find it annoying, since it messes up the output. Such a system wide message should only appear in its log instead of the error output.

Please enhance the code that report the above message, logging it instead of printing it to the standard error.

Many thanks.
Comment 1 William Chen CLA 2014-02-08 01:47:19 EST
Created attachment 239762 [details]
An example of unwanted "Job found still running..."

This is an example command line tool which is based on eclipse application. This tool prints "Job found still running..." to the console, which is unwanted. Asking ECF to fix the bug of ECF jobs not canceled upon exit is one way. But this application is a complex system which depends on numbers of third party plugins, which might not cancel the jobs correctly. Suppressing the message by logging is a good way to avoid the annoyance.
Comment 2 John Arthorne CLA 2014-02-08 23:11:57 EST
This is the comment in code:

// The RuntimeLog.log in its current implementation won't produce a log 
// during this stage of shutdown. For now add a standard error output.
// One the logging story is improved, the System.err output below can be removed:
System.err.println(msg);

I believe this message is useful because it indicates a plugin did not shutdown correctly and this might cause other strange failures.
Comment 3 William Chen CLA 2014-02-10 04:21:08 EST
Yes, it is useful. But I want to make it configurable that the message should be displayed.