[Koha-bugs] [Bug 27421] Porting tools/stage-marc-import.pl to BackgroundJob

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27421

Jonathan Druart <jonathan.druart at bugs.koha-community.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|20.11                       |unspecified
           Assignee|koha-bugs at lists.koha-commun |domm at plix.at
                   |ity.org                     |
                 CC|                            |jonathan.druart at bugs.koha-c
                   |                            |ommunity.org

--- Comment #1 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
Yes, that's the idea.


(In reply to Thomas Klausner from comment #0)
> * Is it neccessary to get the job-size before starting it? If yes, then I
> assume I'll have to first parse the MARC file to get the job size, throwing
> the results away (and this might not work, because we want to do the parsing
> in the background in the first place). Or is there an easy way to get the
> number of records?

I had the same problem with bug 27344. If it get too complicated (or cause perf
issues) I'd say we can consider we are processing 1 file and so its size would
be 1 :)
At least for a first step.

> * When I try to update job_size later, I get an exception (something like
> "job_size not tested")

The accessor is ->size

> * Can I actually use BatchStageMarcRecords in BackgroundJob?

Why not? Did you face a problem?

> * I see I can pass a batch size to BatchStageMarcRecords. But this will not
> result in more forking? This is "only" used to fire the progress callback?

I don't understand the question. Isn't BatchStageMarcRecords used to import the
record? No fork will be done if you are using Koha::BackgroundJob.

> I'm also not sure if the bug tracker or the dev mailing list is the best
> place to discuss my questions?

Here it's perfect :)

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are watching all bug changes.