Периодически ошибка при выполнении запроса

@MirDj · Регистрация: 16.01.2013

Студворк — интернет-сервис помощи студентам

Здравствуйте, уважаемые форумчане.
Есть у меня php файл, который запускается кроном раз в сутки и в нем выполняется запрос, который копирует комментарии из чата в архив. Структура обеих таблиц полностью идентична. И что странно, иногда мне от крона приходит ошибка на почту

Code
1
2
3
4
5
PHP Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '1078458' for key 'PRIMARY'' in /var/www/user/data/www/site.ru/cron/chat.php:10
Stack trace:
#0 /var/www/user/data/www/site.ru/cron/chat.php(10): PDO->query('INSERT INTO `ur...')
#1 {main}
  thrown in /var/www/user/data/www/site.ru/cron/chat.php on line 10

Сам запрос:

PHP
1
$sql->query("INSERT INTO `chatArhive` SELECT * FROM `comments` where news_id = '77278' and dateAdd < (NOW() - INTERVAL 2 DAY)");

Так вот, почему вообще такое может возникать и как с этим бороться?
Какой-то закономерности не наблюдается. Может месяц нормально копироваться, а потом ошибка, а может несколько раз за несколько недель ошибку выдать.
Что это за ошибка я знаю, но вот почему она появляется, не пойму, ведь все должно копироваться из одной таблицы в другую и дубликатов по идее быть не может.

@Jodah · 19.11.2018, 23:11

Надо весь код смотреть, гадать бессмысленно.

@MirDj · 20.11.2018, 06:23 **[ТС]**

Jodah, Так а какой код-то еще нужен? Это и есть весь код. Это запрос в php файле, который раз в сутки запускается кроном и копирует записи из одной таблицы в другую. Структура у них полностью одинаковы:

SQL
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
CREATE TABLE IF NOT EXISTS `chatArhive` (
`id` INT(11) NOT NULL,
  `name` VARCHAR(255) NOT NULL DEFAULT '',
  `email` VARCHAR(255) NOT NULL DEFAULT '',
  `text` mediumtext NOT NULL,
  `news_id` INT(11) NOT NULL DEFAULT '0',
  `quote_id` INT(11) NOT NULL,
  `quoteidFor` INT(11) NOT NULL,
  `usernameFor` VARCHAR(100) NOT NULL,
  `useridFor` INT(11) NOT NULL,
  `plus` INT(11) NOT NULL,
  `minus` INT(11) NOT NULL,
  `user_id` INT(11) NOT NULL,
  `dateAdd` datetime NOT NULL,
  `timeEdit` VARCHAR(100) NOT NULL
) ENGINE=MyISAM AUTO_INCREMENT=1083705 DEFAULT CHARSET=utf8;
 
--
-- Индексы сохранённых таблиц
--
 
--
-- Индексы таблицы 'chatArhive`
--
ALTER TABLE `chatArhive`
 ADD PRIMARY KEY (`id`), 
ADD KEY `news_id` (`news_id`),
 ADD KEY `quote_id_news_id` (`quote_id`,`news_id`),
 ADD KEY `news_id_dateAdd` (`news_id`,`dateAdd`), 
ADD KEY `id_news_id_quote_id` (`id`,`news_id`,`quote_id`), 
ADD KEY `quote_id` (`quote_id`), 
ADD KEY `user_id` (`user_id`), 
ADD KEY `news_id_user_id` (`news_id`,`user_id`), 
ADD FULLTEXT KEY `text` (`text`);
 
--
-- AUTO_INCREMENT для таблицы `chatArhive`
--
ALTER TABLE `chatArhive`
MODIFY `id` INT(11) NOT NULL AUTO_INCREMENT,AUTO_INCREMENT=1083705;

@Jodah · 20.11.2018, 07:15

Сообщение от MirDj

INTERVAL 2 DAY

Если раз в сутки, почему интервал 2 дня?

@MirDj · 20.11.2018, 08:14 **[ТС]**

Jodah, Мне надо чтобы в чате содержимое сохранялось за последние двое суток, а не обнулялось полностью.
Предвижу вопрос по поводу того чтобы запускать скрипт раз в двое суток, но там еще выполняются и другие операции, которые нужно делать именно раз в сутки. Отдельно делать задание и файл для одного запроса?

@Jodah · 20.11.2018, 08:59

Сообщение от MirDj

Мне надо чтобы в чате содержимое сохранялось за последние двое суток, а не обнулялось полностью.

А перед записью в архив вы удаляете содержимое этого архива? Т.е. INSERT-запрос происходит когда архивная таблица пустая или там что-то лежит?

@MirDj · 20.11.2018, 09:02 **[ТС]**

Jodah, А архивной таблице записи добавляются при каждой работе крона и там хранятся именно как архив.

@Jodah · 20.11.2018, 09:19

Сообщение от MirDj

Мне надо чтобы в чате содержимое сохранялось за последние двое суток

Стоп, а причём тут чат? Вы указали интервал для архива, а не для чата.

Получается, скрипт через каждые 24 часа пытается перенести в архивную таблицу записи, сделанные за последние 48 часов. Отсюда и дубли.

Сообщение от MirDj

Отдельно делать задание и файл для одного запроса?

Просто заменить интервал на 1 день.

@MirDj · 20.11.2018, 09:21 **[ТС]**

Jodah,

Сообщение от Jodah

Вы указали интервал для архива, а не для чата.

Как же так? Вот условие выборки в таблице чата внутри запроса при инсерте

Сообщение от MirDj

SELECT * FROM `comments` where news_id = '77278' and dateAdd < (NOW() - INTERVAL 2 DAY)

@Jodah · 20.11.2018, 09:38

MirDj, не, я про другое. Вы написали:

Сообщение от MirDj

надо чтобы в чате содержимое сохранялось за последние двое суток

Но тот код, который вы указали в 1-ом посте, относится к сохранению записей в архиве, а не в чате.

В общем, нужно либо интервал сделать 1 день, либо запускать скрипт раз в 2 дня. Иначе скрипт пытается бекапить записи, которые до этого уже бекапил за день до этого.

@MirDj · 20.11.2018, 09:41 **[ТС]**

Сообщение от Jodah

Но тот код, который вы указали в 1-ом посте, относится к сохранению записей в архиве, а не в чате.

Так у меня и стоит задача:
Раз в сутки добавлять в таблицу с архивом данные из чата, но чтобы в чате оставались записи за последние двое суток.

@Jodah · 20.11.2018, 16:04

MirDj, это понятно, но удаление записей из чата вы ведь отдельным запросом делаете. И в этом запросе уже указывается интервал 2 дня.

@MirDj · 20.11.2018, 16:12 **[ТС]**

Сообщение от Jodah

но удаление записей из чата вы ведь отдельным запросом делаете

Да, вот у меня все 3 запроса:
Копирую в архив

PHP
1
$sql->query("INSERT INTO `chatArhive` SELECT * FROM `".MY_PREFIX."_comments` where news_id = '77278' and dateAdd < (NOW() - INTERVAL 2 DAY)");

Удаляю комментарии

PHP
1
$delCom = (int) $sql->exec("delete from `comments` where news_id = '77278' and quote_id = '0' and dateAdd < (NOW() - INTERVAL 2 DAY)");

И удаляю ответы (все отдельно чтобы счетчики обновить)

PHP
1
$delQuote = (int) $sql->exec("delete from `comments` where news_id = '77278' and quote_id != '0' and  dateAdd < (NOW() - INTERVAL 2 DAY)");

@Jodah · 20.11.2018, 16:17

MirDj, вот. Теперь в копировании указываем интервал 1 день, а удаление оставляем как есть. И скрипт должен будет работать как вам нужно.

@MirDj · 20.11.2018, 16:23 **[ТС]**

Jodah, Благодарю. Буду проверять, если все будет как надо, отмечу как лучший ответ...

@MirDj · 22.11.2018, 06:15 **[ТС]**

Jodah, Сегодня снова получил эту же ошибку (((

@Jodah · 22.11.2018, 09:28

MirDj,

Сообщение от MirDj

and dateAdd < (NOW() - INTERVAL 2 DAY)

Здесь вроде > должно быть. Т.е. архивировать, если дата создания больше вчерашней.

У этого поля формат datetime, да?

@MirDj · 22.11.2018, 09:29 **[ТС]**

Сообщение от Jodah

У этого поля формат datetime, да?

Да, datetime

@Jodah · 22.11.2018, 09:30

MirDj, ок, тогда попробуйте с >.

@MirDj · 22.11.2018, 09:33 **[ТС]**

Сообщение от Jodah

Здесь вроде > должно быть.

Так по логике, если я все правильно понимаю, то скопировать сообщения, где дата добавления раньше чем сегодня на 2 дня. Разве не так?

Добавлено через 59 секунд
Т.е должны скопироваться все, кроме тех что добавлены за последние 2 дня

Новые блоги и статьи Все статьи Все блоги /
Отчёт о затраченных материалах за определенный период с макетом печатной формы Maks 21.04.2026 Отчёт из решения ниже размещён в конфигурации КА2. Задача: разработка отчёта по затраченным материалам за определённый период, с возможностью вывода печатной формы отчёта с шапкой и подвалом. В. . .	Отчёт о спецтехнике находящейся в ремонте Maks 20.04.2026 Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .	Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)" Hrethgir 19.04.2026 Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .	Запрет удаления строк ТЧ документа при определённом условии Maks 19.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "Аккумуляторы", разработанного в конфигурации КА2. У данного документа есть ТЧ, в которой в зависимости от прав доступа. . .
Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.

@Jodah 3900 / 3238 / 1353 Регистрация: 01.08.2012 Сообщений: 10,914
	19.11.2018, 23:11
	Надо весь код смотреть, гадать бессмысленно. 0

@MirDj 3 / 3 / 2 Регистрация: 16.01.2013 Сообщений: 471
	20.11.2018, 08:14 [ТС]
	Jodah, Мне надо чтобы в чате содержимое сохранялось за последние двое суток, а не обнулялось полностью. Предвижу вопрос по поводу того чтобы запускать скрипт раз в двое суток, но там еще выполняются и другие операции, которые нужно делать именно раз в сутки. Отдельно делать задание и файл для одного запроса? 0

@MirDj 3 / 3 / 2 Регистрация: 16.01.2013 Сообщений: 471
	20.11.2018, 09:02 [ТС]
	Jodah, А архивной таблице записи добавляются при каждой работе крона и там хранятся именно как архив. 0

@Jodah 3900 / 3238 / 1353 Регистрация: 01.08.2012 Сообщений: 10,914
	20.11.2018, 16:04
	MirDj, это понятно, но удаление записей из чата вы ведь отдельным запросом делаете. И в этом запросе уже указывается интервал 2 дня. 0

@Jodah 3900 / 3238 / 1353 Регистрация: 01.08.2012 Сообщений: 10,914
	20.11.2018, 16:17
	MirDj, вот. Теперь в копировании указываем интервал 1 день, а удаление оставляем как есть. И скрипт должен будет работать как вам нужно. 1

@MirDj 3 / 3 / 2 Регистрация: 16.01.2013 Сообщений: 471
	20.11.2018, 16:23 [ТС]
	Jodah, Благодарю. Буду проверять, если все будет как надо, отмечу как лучший ответ... 0

@MirDj 3 / 3 / 2 Регистрация: 16.01.2013 Сообщений: 471
	22.11.2018, 06:15 [ТС]
	Jodah, Сегодня снова получил эту же ошибку ((( 0

@Jodah 3900 / 3238 / 1353 Регистрация: 01.08.2012 Сообщений: 10,914
	22.11.2018, 09:30
	MirDj, ок, тогда попробуйте с `>`. 0