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

@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 дня

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536 Одним из. . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .

@Jodah 3899 / 3237 / 1353 Регистрация: 01.08.2012 Сообщений: 10,911
	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 3899 / 3237 / 1353 Регистрация: 01.08.2012 Сообщений: 10,911
	20.11.2018, 16:04
	MirDj, это понятно, но удаление записей из чата вы ведь отдельным запросом делаете. И в этом запросе уже указывается интервал 2 дня. 0

@Jodah 3899 / 3237 / 1353 Регистрация: 01.08.2012 Сообщений: 10,911
	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 3899 / 3237 / 1353 Регистрация: 01.08.2012 Сообщений: 10,911
	22.11.2018, 09:30
	MirDj, ок, тогда попробуйте с `>`. 0