[Koha-bugs] [Bug 35920] Centralize code from workers

2024-10-15 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #21 from Jonathan Druart --- (In reply to Tomás Cohen Arazi (tcohen) from comment #20) > I like Jonathan's code, but we really need to decide if we want to keep > Rabbit or not. I don't think we have discussed/voted it at

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-10-15 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #20 from Tomás Cohen Arazi (tcohen) --- I like Jonathan's code, but we really need to decide if we want to keep Rabbit or not. I don't think we have discussed/voted it at all. I understand RabbitMQ has some advocates. I'm

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-04-16 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 Jonathan Druart changed: What|Removed |Added Status|NEW |In Discussion -- You are

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-04-16 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #19 from Jonathan Druart --- (In reply to Martin Renvoize from comment #18) > I like the direction of travel here too.. but the NEW status doesn't really > highlight if it's ready for testing.. feedback.. or what? It's f

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-04-16 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 Martin Renvoize changed: What|Removed |Added CC||martin.renvoize@ptfs-europ

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-25 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #17 from Marcel de Rooy --- (In reply to Jonathan Druart from comment #16) > No opinion on it? I happened to ignore the whole patch when I wrote the others. But viewing diagonally, I would like to go that direction. So y

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-25 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #16 from Jonathan Druart --- No opinion on it? -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailin

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-25 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #15 from Marcel de Rooy --- (In reply to Jonathan Druart from comment #14) > (In reply to Marcel de Rooy from comment #13) > > Comment on attachment 162951 [details] [review] [review] > > Bug 35920: Centralize worker code

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-25 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #14 from Jonathan Druart --- (In reply to Marcel de Rooy from comment #13) > Comment on attachment 162951 [details] [review] > Bug 35920: Centralize worker code > > Abandoning it. Asked for feedback, received none, etc.

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-22 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 Marcel de Rooy changed: What|Removed |Added Attachment #162952|0 |1 is obsolete|

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-22 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 Marcel de Rooy changed: What|Removed |Added Attachment #162951|0 |1 is obsolete|

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-22 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 Marcel de Rooy changed: What|Removed |Added Assignee|m.de.r...@rijksmuseum.nl|koha-b...@lists.koha-commun

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #12 from Jonathan Druart --- (In reply to Marcel de Rooy from comment #11) > (In reply to Jonathan Druart from comment #1) > > This is WIP. We need bug 35819 pushed first, then to clean this, it is not > > very nice. But

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #11 from Marcel de Rooy --- (In reply to Jonathan Druart from comment #1) > This is WIP. We need bug 35819 pushed first, then to clean this, it is not > very nice. But at least it will prevent the code to be duplicated.

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 Marcel de Rooy changed: What|Removed |Added CC||n...@bywatersolutions.com

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #10 from Marcel de Rooy --- Created attachment 162952 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=162952&action=edit Bug 35920: Use BackgroundWorker in es_indexer_daemon This needs incorporating the

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #9 from Marcel de Rooy --- Created attachment 162951 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=162951&action=edit Bug 35920: Centralize worker code Test plan: Confirm that running jobs still works

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 Marcel de Rooy changed: What|Removed |Added Patch complexity|--- |Small patch -- You are re

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 Marcel de Rooy changed: What|Removed |Added Depends on||33898 Referenced Bugs: h

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #8 from Marcel de Rooy --- Giving this a try on top of 33898 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lis

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-03-08 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 Marcel de Rooy changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |m.de.r...@rijksmuseum.nl

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-02-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #7 from Marcel de Rooy --- (In reply to Tomás Cohen Arazi from comment #6) > (In reply to Jonathan Druart from comment #5) > > (In reply to Tomás Cohen Arazi from comment #3) > > > I like the concept. > > > > > > Given t

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-02-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 Marcel de Rooy changed: What|Removed |Added Blocks||35092 Referenced Bugs: h

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-02-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #6 from Tomás Cohen Arazi --- (In reply to Jonathan Druart from comment #5) > (In reply to Tomás Cohen Arazi from comment #3) > > I like the concept. > > > > Given the discussion Marcel is driving around rabbit vs. polli

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-01-31 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #5 from Jonathan Druart --- (In reply to Tomás Cohen Arazi from comment #3) > I like the concept. > > Given the discussion Marcel is driving around rabbit vs. polling, why not > create some base class and then different

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-01-31 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #4 from Marcel de Rooy --- (In reply to Tomás Cohen Arazi from comment #3) > I like the concept. > > Given the discussion Marcel is driving around rabbit vs. polling, why not > create some base class and then different c

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-01-31 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #3 from Tomás Cohen Arazi --- I like the concept. Given the discussion Marcel is driving around rabbit vs. polling, why not create some base class and then different classes implementing each communication mechanism? Tha

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-01-26 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 Jonathan Druart changed: What|Removed |Added CC||dc...@prosentient.com.au,

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-01-26 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #2 from Jonathan Druart --- Created attachment 161544 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=161544&action=edit Bug 35920: Centralize code from workers -- You are receiving this mail because:

[Koha-bugs] [Bug 35920] Centralize code from workers

2024-01-26 Thread bugzilla-daemon
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35920 --- Comment #1 from Jonathan Druart --- This is WIP. We need bug 35819 pushed first, then to clean this, it is not very nice. But at least it will prevent the code to be duplicated. -- You are receiving this mail because: You are t