Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 335524

Summary:

include task id in error message when synchronization fails

Product:

z_Archived

Reporter:

Thomas Ehrnhoefer <thomas.ehrnhoefer>

Component:

Mylyn

Assignee:

Thomas Ehrnhoefer <thomas.ehrnhoefer>

Status:

RESOLVED FIXED

QA Contact:

Severity:

trivial

Priority:

CC:

steffen.pingel

Version:

unspecified

Keywords:

contributed

Target Milestone:

3.5

Hardware:

OS:

Windows 7

Whiteboard:

Attachments:

Description	Flags
patch	none
patch v2	none
patch v3	none
patch v3.1	steffen.pingel: iplog+

Description Thomas Ehrnhoefer

2011-01-26 17:59:24 EST

Currently a failure in synchronizing related tasks will lead to "Synchronization failed" in the error log. The details list the error message from the server, which does not necessarily have more information on what the task is either.
This is not very helpful, as the user has no idea what task failed to synchronize.

Comment 1 Thomas Ehrnhoefer

2011-01-26 18:00:36 EST

Created attachment 187696 [details]
patch

minimalist approach to add some information. We might want the full Url though, or the repository label as well.

Comment 2 Thomas Ehrnhoefer

2011-01-26 18:02:47 EST

Created attachment 187697 [details]
patch v2

this one add repo label to the error message

Comment 3 Steffen Pingel

2011-01-26 18:23:15 EST

Makes sense but I think the task should be shown by default since the ID is less meaningful.

Comment 4 Thomas Ehrnhoefer

2011-01-26 18:53:46 EST

Created attachment 187702 [details]
patch v3

updated

Comment 5 Thomas Ehrnhoefer

2011-01-26 18:54:35 EST

(In reply to comment #3)
> Makes sense but I think the task should be shown by default since the ID is less
> meaningful.
I updated the patch, it now logs
Synchronization of taskKey [repo label] failed

Comment 6 Thomas Ehrnhoefer

2011-01-26 18:55:19 EST

Created attachment 187703 [details]
patch v3.1

Comment 7 Steffen Pingel

2011-02-01 23:24:24 EST

Thanks! I have applied the patch.

Comment 8 Steffen Pingel

2011-02-01 23:24:48 EST

Closing.