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

Bug 260366

Summary:	[EFS] Blocking file transfers in EFS
Product:	[Eclipse Project] Platform	Reporter:	Mathias Stümpert <mathias.stuempert>
Component:	Resources	Assignee:	Sergey Prigogin <eclipse.sprigogin>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	P5	CC:	aog-ecl, eclipse.sprigogin, edpeur, mknauer, reuillon, Szymon.Brandys
Version:	3.4	Keywords:	helpwanted
Target Milestone:	4.6 M1
Hardware:	All
OS:	All
See Also:	https://git.eclipse.org/r/52026 https://git.eclipse.org/c/platform/eclipse.platform.resources.git/commit/?id=7ae8635247606c7dbd44aae8cbdabf845dc90286
Whiteboard:

Description Mathias Stümpert

2009-01-08 05:14:48 EST

Having a look at FileStore#transferStreams it seems that this method synchronizes on the buffer object that is declared to be static final. That means that transfers can effectively not happen in parallel which is a pity.

Comment 1 Szymon Brandys

2009-01-08 08:10:31 EST

Right... this is also mentioned in the comment right before the synchronized block. Patches are always welcome. Would you like work on that Mathias? I think that the fix is not a hard one.

Comment 2 Mathias Stümpert

2009-01-08 08:27:17 EST

> Right... this is also mentioned in the comment right before the synchronized
> block. Patches are always welcome. Would you like work on that Mathias? I think
> that the fix is not a hard one.

The hard thing is always finding some time ;-)

Think it is not that urgent, so I can take over. So feel free to reassign to me.

Comment 3 John Arthorne

2009-01-12 00:02:00 EST

This optimization is there to reduce garbage creation, since the resource API doesn't support concurrent writes anyway. Someone using FileStore directly may want to perform writes concurrently I suppose.

Comment 4 Szymon Brandys

2009-01-12 05:57:01 EST

I'm lowering the priority and marking as helpwanted.

Comment 5 John Arthorne

2012-03-13 08:45:53 EDT

*** Bug 372415 has been marked as a duplicate of this bug. ***

Comment 6 Eclipse Genie

2015-07-15 22:31:49 EDT

New Gerrit change created: https://git.eclipse.org/r/52026

Comment 7 Eclipse Genie

2015-07-24 18:00:24 EDT

Gerrit change https://git.eclipse.org/r/52026 was merged to [master].
Commit: http://git.eclipse.org/c/platform/eclipse.platform.resources.git/commit/?id=7ae8635247606c7dbd44aae8cbdabf845dc90286

Comment 8 Sergey Prigogin

2015-07-24 19:06:05 EDT