search

onlinerby / onliner-second-api

1 stars 1 forks source link

GET /shop/offers #3

Closed tronicsby closed 2 years ago

tronicsby commented 6 years ago

Добрый день Было бы неплохо в ответе передавать еще и цену на товар.

sidorovich commented 6 years ago

Предполагалось, что объявления на уцененные товары уже хранятся в каком-либо виде в вашей системе для автоматизации продаж, и идентификатора объявления вполне достаточно для связки между нашими системами. Не могли бы Вы подробнее описать нам почему возникает необходимость в получении стоимости от нас и как в принципе Вы управляете данными в объявлении? Это помогло бы нам лучше понимать Ваши потребности.

tronicsby commented 6 years ago

В нашей базе не хранится история цен. Без этого, при выгрузке, мне прийдется сохранять id и цену в отдельной таблице. Чтобы по новой не заливать товар, в котором не изменилась цена.

tronicsby commented 6 years ago

Сделал с помощью таблицы :)

tronicsby commented 2 years ago

Добрый день. Приходит пустой ответ. Ошибок нет. Хотя товары размещены.

sidorovich commented 2 years ago

Уточните, пожалуйста, номер вашего магазина, а также какой HTTP код ответа получаете при пустом ответе

tronicsby commented 2 years ago

HTTP/1.1 500 Internal Server Error Server: nginx Content-Type: text/html; charset=UTF-8 Transfer-Encoding: chunked Connection: keep-alive Keep-Alive: timeout=15 Cache-Control: no-cache, private Date: Tue, 24 May 2022 07:56:32 GMT Access-Control-Allow-Origin: Access-Control-Allow-Credentials: true

Магазин №1689

tronicsby commented 2 years ago

но если я отправляю неверный токен то все нормально. Приходит ошибка "Требуется авторизация"

sidorovich commented 2 years ago

видим... [ERROR] second-api fpm-fcgi "GET /shop/offers" Symfony\Component\Debug\Exception\FatalErrorException: Allowed memory size of 536870912 bytes exhausted (tried to allocate 1859584 bytes) будем изучать чего у вас так много данных

sidorovich commented 2 years ago

у ваших объявлений просто огромное количество поднятий, около 41 тысячи. данный роут делался в условиях ограниченных ресурсов и в нем нет пагинации - будем дорабатывать в будущем. в текущий момент у вас 149 активных объявлений, у половины из них по 41 тысячи поднятий. как часто вы выгружаете нам данные? просто у нас накапливаются данные по изменениям, даже если вы передаете одну и ту же информацию в рамках одного и того же id объявления.

sidorovich commented 2 years ago

можете подсказать ваш IP адрес, с которого идут обращения к нашему API?

sidorovich commented 2 years ago

мы почистили ненужные логи. жду вашего ответа по IP-адресу и комментарии о частоте выгрузки. будем думать как сделать так, чтобы проблема не появлялась больше

tronicsby commented 2 years ago

93.125.48.69

tronicsby commented 2 years ago

будем выгружать 1-2 раза в сутки

sidorovich commented 2 years ago

спасибо! это нам очень сильно поможет. как минимум у нас будет больше времени, чтобы придумать нормальное решение

tronicsby commented 2 years ago

может тогда сразу уберете контроль по цене? Чтобы можно было и увеличивать. Сейчас реализовано только на понижение. Это важный момент и мы с ним постоянно сталкиваемся. Сами понимаете. Бывают моменты когда очень сильно вырастает курс....

sidorovich commented 2 years ago

я закину этот запрос, но это не техническая проблема, а зафиксированное бизнес-требование. пересмотр его потребует больше ресурсов

tronicsby commented 2 years ago

попробуйте получить ошибки по id - vmYX приходит очень большой массив много дублей посмотрите на примере товара с id - 900427 повтор 37 раз, что "Товар не найден"

sidorovich commented 2 years ago

  1. Фикс проблемы выгружен на прод. 500ая ошибка больше не должна проявляться при получении списка объявлений, так как устранена причина накопления излишнего объема информации. Будем наблюдать за поведением наших обработчиков в течении ближайшей недели.

  2. По поводу изменения стоимости в большую сторону бизнес ответил негативно, так как монетизация данного продукта основана на проценте от указанной в объявлении стоимости, и это позволит магазинам злоупотреблять размещая объявления по заведомо ложной низкой стоимости, а после списания за услугу увеличивать стоимость. Проработка вариантов, для устранения данной проблемы требует много ресурсов на нашей стороне. Будем продолжать увеличивать стоимость на объявления в частном порядке, через обращения в наш отдел поддержки. Нам удобно получать новые стоимости в виде CSV-файла с указанием id объявления и новой стоимости.

Если проблема не воспроизводится и вопросов больше нет, то прошу закрыть данный тикет. Новые вопросы или проблемы будем обсуждать уже в рамках других. Спасибо!

tronicsby commented 2 years ago

Посмотрите еще ошибки. Последнее мое сообщение.