Bugzilla – Attachment 126224 Details for
Bug 29130
Holds queue is empty while holds queue builder is running
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 29130: Make the temp tables persistent so unit tests can pass
Bug-29130-Make-the-temp-tables-persistent-so-unit-.patch (text/plain), 2.65 KB, created by
Kyle M Hall
on 2021-10-13 16:06:32 UTC
(
hide
)
Description:
Bug 29130: Make the temp tables persistent so unit tests can pass
Filename:
MIME Type:
Creator:
Kyle M Hall
Created:
2021-10-13 16:06:32 UTC
Size:
2.65 KB
patch
obsolete
>From 767d09c1e08c71f1704ab63370f54eaa20d5f26e Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Wed, 13 Oct 2021 16:06:09 +0000 >Subject: [PATCH] Bug 29130: Make the temp tables persistent so unit tests can > pass > >--- > C4/HoldsQueue.pm | 8 ++------ > installer/data/mysql/kohastructure.sql | 14 ++++++++++++++ > 2 files changed, 16 insertions(+), 6 deletions(-) > >diff --git a/C4/HoldsQueue.pm b/C4/HoldsQueue.pm >index c353f9ee9b8..28c0a5d8e50 100644 >--- a/C4/HoldsQueue.pm >+++ b/C4/HoldsQueue.pm >@@ -167,10 +167,8 @@ Top level function that turns reserves into tmp_holdsqueue and hold_fill_targets > sub CreateQueue { > my $dbh = C4::Context->dbh; > >- $dbh->do("DROP TABLE IF EXISTS tmp_holdsqueue_builder"); >- $dbh->do("DROP TABLE IF EXISTS hold_fill_targets_builder"); >- $dbh->do("CREATE TABLE tmp_holdsqueue_builder LIKE tmp_holdsqueue"); >- $dbh->do("CREATE TABLE hold_fill_targets_builder LIKE hold_fill_targets"); >+ $dbh->do("DELETE FROM tmp_holdsqueue_builder"); >+ $dbh->do("DELETE FROM hold_fill_targets_builder"); > > my $total_bibs = 0; > my $total_requests = 0; >@@ -222,8 +220,6 @@ sub CreateQueue { > $dbh->do("DELETE FROM hold_fill_targets"); > $dbh->do("INSERT INTO tmp_holdsqueue SELECT * FROM tmp_holdsqueue_builder"); > $dbh->do("INSERT INTO hold_fill_targets SELECT * FROM hold_fill_targets_builder"); >- $dbh->do("DROP TABLE tmp_holdsqueue_builder"); >- $dbh->do("DROP TABLE hold_fill_targets_builder"); > $dbh->commit() unless $do_not_lock; > } > >diff --git a/installer/data/mysql/kohastructure.sql b/installer/data/mysql/kohastructure.sql >index 95a3735906c..df791b12286 100644 >--- a/installer/data/mysql/kohastructure.sql >+++ b/installer/data/mysql/kohastructure.sql >@@ -2631,6 +2631,13 @@ CREATE TABLE `hold_fill_targets` ( > ) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci; > /*!40101 SET character_set_client = @saved_cs_client */; > >+-- >+-- Table structure for table `hold_fill_targets_builder` is a duplicate of `hold_fill_targets` >+-- >+ >+DROP TABLE IF EXISTS `hold_fill_targets_builder`; >+CREATE TABLE hold_fill_targets_builder LIKE tmp_holdsqueue; >+ > -- > -- Table structure for table `housebound_profile` > -- >@@ -5110,6 +5117,13 @@ CREATE TABLE `tmp_holdsqueue` ( > ) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci; > /*!40101 SET character_set_client = @saved_cs_client */; > >+-- >+-- Table structure for table `tmp_holdsqueue_builder` is a duplicate of `tmp_holdsqueue` >+-- >+ >+DROP TABLE IF EXISTS `tmp_holdsqueue_builder`; >+CREATE TABLE tmp_holdsqueue_builder LIKE tmp_holdsqueue; >+ > -- > -- Table structure for table `transport_cost` > -- >-- >2.20.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 29130
:
125427
|
125430
|
125431
|
125432
|
126215
|
126224
|
126225
|
126227