apidoc during build step is terribly slow

[dgrove]

There are two parts to this:

1. apidoc appears to run when it doesn't need to - there must be something wrong with its dependencies.
2. apidoc itself is extremely slow these days.

[dgrove]

Added this to the M1 milestone.

[sethladd]

When profiling apidoc, please check IO performance. We used to generate "dynamic" docs, which results in less bytes written to the file system. However, the downside is that it results in slower page loads. We now generate "static" docs, so our api.dartlang.org pages load much faster now.

Also, what does "terribly slow" mean?

[jmesserly]

As Lars said: "
It now takes more than 20 minutes to generate the api docs.
Please look into it. The task is part of tools/build.py -m debug.
Using the release VM "only" takes 1:39. It might be a VM issue.
"

20 minute is terribly slow :)

In general I don't think we want to run tools on the debug VM.

[DartBot]

This comment was originally written by [email protected]

---

Does anyone want to speak in favor of having build.py run apidoc on the checked-mode VM?

My understanding is that

1. build.py runs apidoc to expose apidoc breaks,
2. those are less common now that apidoc uses dart2js machinery,
3. we aren't aware of needing coverage for breakage specific to checked mode, though it could happen.

[munificent]

I'm OK with disabling the apidoc build step in debug mode.

[sethladd]

Did this get resolved? I remember the VM guys tweaks some code that sped this up.

---

cc @iposva-google.

[dgrove]

Some improvements have been made here, but there's still more to do for M2.

---

cc @rakudrama.
Set owner to @dgrove.
Removed this from the M1 milestone.
Added this to the M2 milestone.

[anders-sandholm]

Removed this from the M2 milestone.
Added this to the M3 milestone.

[anders-sandholm]

Removed this from the M3 milestone.
Added this to the M4 milestone.

[larsbak]

Removed this from the M4 milestone.
Added this to the M5 milestone.

[ricowind]

We don't do that anymore right, should this be closed

[anders-sandholm]

Removed Area-Build label.
Added Area-Infrastructure label.

[kasperl]

Removed this from the M5 milestone.
Added this to the 1.6 milestone.

[kasperl]

Removed this from the 1.6 milestone.
Added Oldschool-Milestone-1.6 label.

[kasperl]

Removed Oldschool-Milestone-1.6 label.

[ricowind]

Added AssumedStale label.