Task #11254 (closed)
Opened 6 years ago
Closed 6 years ago
Bug: Importer does not show any import metrics
| Reported by: | rleigh | Owned by: | jburel |
|---|---|---|---|
| Priority: | minor | Milestone: | 5.x |
| Component: | Import | Version: | 5.0.0-beta1 |
| Keywords: | FS | Cc: | ux@…, fs@… |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | n.a. |
| Sprint: | n.a. |
Description
The 4.4 importer showed the total time elapsed during import. The 5.x importer does not display any equivalent metrics.
Note that the import log only shows timestamps for the duration of the metadata import; it does not start or stop logging at the real start or end time so this is not useful for profiling import performance.
Change History (6)
comment:1 Changed 6 years ago by jburel
comment:2 Changed 6 years ago by jburel
- Cc ux@… fs@… added
comment:3 Changed 6 years ago by rleigh
We can record the real (wall-clock) time at the start quite easily and compute the elapsed time from that. It doesn't need to involve any log files; we can compute it directly and update it quite easily. We used to have this information in the old importer.
comment:4 Changed 6 years ago by jburel
of course we can compute it. Remember that now you can close the importer/session when the upload is complete so the "import time" as it was displayed before is no longer valid. It was not an information we wanted to keep i.e. time of an import queue.
This will require more feedback.
comment:5 Changed 6 years ago by jamoore
- Milestone changed from Unscheduled to OMERO-5
Moving into OMERO-5. That milestone will likely be renamed to "beta1" and closed and which point a better place for this ticket will need to be found.
comment:6 Changed 6 years ago by jburel
- Resolution set to invalid
- Status changed from new to closed
Upload size and time are indicated. No plan to add more metrics for now
The only info we can display is the upload time. The "import" time per say will have to be extracted from the log file or returned by an event.
We did not really want to introduce too much confusion.