Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 347477 - Only load p2 artifact repositories if an artifact is missing in local cache
Summary: Only load p2 artifact repositories if an artifact is missing in local cache
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Tycho (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Tobias Oberlies CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 393004
Blocks:
  Show dependency tree
 
Reported: 2011-05-27 11:47 EDT by Tobias Oberlies CLA
Modified: 2021-04-28 16:55 EDT (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Tobias Oberlies CLA 2011-05-27 11:47:07 EDT 
Tycho caches all p2 artifacts in the local Maven repository. The LocalMavenRepository implementation is able to answer the question if a specific IArtifactKey is available. Therefore the p2 resolver could postpone the loading of the (remote) p2 artifact repositories to the point where it finds a locally unavailable artifact in the resolved target platform. The target platform resolution is solely based on p2 metadata, so this is possible.

This cuts down the typical, total number of HEAD http requests to half (-> no more requests to the artifact repos), with bug 347403 fixed possibly even to a much lower percentage. The number of http round trips is the reason why "Adding repository http://download.eclipse.org/releases/helios" takes so long.
Comment 1 Tobias Oberlies CLA 2011-06-06 06:51:44 EDT 
Changing severity to normal, because this is a serious performance bug. Scheduling for 0.13
Comment 2 Tobias Oberlies CLA 2011-08-16 03:57:34 EDT 
I probably won't have time to fix this for 0.13...
Comment 3 Tobias Oberlies CLA 2013-03-15 12:24:31 EDT 
This behaviour is already implemented in the (currently still partial) patch for bug 393004.
Comment 4 Tobias Oberlies CLA 2013-06-24 06:59:44 EDT 
This performance improvement comes with this change: https://git.eclipse.org/r/#/c/9333/
Comment 5 Tobias Oberlies CLA 2013-07-08 05:33:34 EDT 
This problem is fixed as part of this larger commit: http://git.eclipse.org/c/tycho/org.eclipse.tycho.git/commit/?id=9fa0c707522eb0dcf37be2cec4d6a99d6c9bb484