Conversation
|
Shortly after we moved to Go 1.5, I looked into why compiles too so long. I found that a significant amount of the time was spent building sqlite. The Go compilation was parallelized, but the C compilation was not since sqlite is one big file. Removing sqlite will probably have a beneficial impact on build times. |
|
(That is, the time to build Docker, not the |
|
I remember we discussed this before, and there was some reluctance, but On 17 Aug 2016 5:59 a.m., "Aaron Lehmann" notifications@github.com wrote:
|
|
See this issue #22146 |
|
ping @cpuguy83 ptal |
|
Oh man, major +1 here. |
|
oh! I miss the migration tool, that is really awesome! Would be great if we had that under the "docker" organization - but guess we won't be able to ship it as an image (like the v1.10 migration tool). FWIW, once we're in docs review, I think we should have it added to https://github.com/docker/docker/blob/master/docs/migration.md |
|
@thaJeztah I think we can incorporate that to migrator tool. |
|
I'm also definitely +1 on that 😻 |
|
@tonistiigi also pointed that we should notify people that they should run migration tool. Not sure how to make that behavior opt-out though. |
|
Nice! @jstarks FYI |
|
btw compile time goes from 50s to 40s on my machine without sqlite. |
Signed-off-by: Alexander Morozov <lk4d4@docker.com>
Signed-off-by: Alexander Morozov <lk4d4@docker.com>
|
I guess my only real concern is that there is still a large contingent of people running docker 1.9 due to build caching issues in >= 1.10. |
|
@cpuguy83 what do you think? Should we move this to 1.14? Or don't do it at all? |
|
@cpuguy83 or shall we document that people from 1.9 should always use 1.10, perhaps we should check if we document that we support upgrading and skipping versions? If so, then I think we can move forward |
|
@thaJeztah Do people read docs before upgrading? |
|
@thaJeztah @cpuguy83 yeah, let's wait for a release with caching improvements, so people can upgrade. I'll close this for now. |
|
Thanks! |
fixes #22146
relates to #16032
- What I did
Removed sqlite dependency.
- How I did it
Removed every usage of graphdb.
- How to verify it
Compile daemon
- Description for the changelog
Dependency from sqlite removed, so links upgrade from 1.9 does not work. External migration tool should be used if necessary.
- A picture of a cute animal (not mandatory but encouraged)

I also wrote tool for migration: https://github.com/LK4D4/linksmigrator