INNER JOIN на 2 таблицы возвращает неправильные значения

Question

Вот мой SQL запрос:

SELECT SUM(amz_event_shipment_items.quantity), 
     amz_event_shipment_items.seller_sku 

FROM amz_event_shipment_items 

INNER JOIN amz_event_fees   ON amz_event_shipment_items.id = amz_event_fees.shipment_item_id 
INNER JOIN amz_shipment_events ON amz_shipment_events.id = amz_event_shipment_items.shipment_event_id 

WHERE amz_event_fees.currency = 'USD' 
     AND amz_shipment_events.event_type <> 'RefundEvent' 
     AND amz_shipment_events.posted_date BETWEEN '2016-5-1 07:00:00' AND '2016-5-7 06:59:59' 

GROUP BY amz_event_shipment_items.seller_sku 

Но возвращаемые значения слишком высоки ... Не может иметь смысл для меня ...

Я что-то пропустил?

Редактировать

Many shipment_events for each date 

Each shipment_event HAS MANY shipment_item/BELONGS TO ONE event 

Each shipment_item HAS MANY shipment_fee/BELONGS TO ONE item 

Answer 1

Это менее ответ, но приложение. Если я правильно понял, ваш запрос вернул неправильные результаты, но был достаточно быстрым, тогда как мои (с предложениями EXISTS) вернули правильные результаты, но очень медленны.

Таким образом, задача устранения дубликатов занимает слишком много времени.

Две идеи здесь:

Первая идея: Исключите дублирует сразу

Вместо joning на сборы, мы собираем взносы до вступления:

select 
    sum(i.quantity), 
    i.seller_sku 
from amz_event_shipment_items i 
join -- join with only one record per ID to substitute an EXISTS clause 
(
    select distinct shipment_item_id 
    from amz_event_fees 
    where f.currency = 'USD' 
) f on f.shipment_item_id = i.id 
and exists 
(
    select * 
    from amz_shipment_events e 
    where e.event_type <> 'RefundEvent' 
    and e.posted_date between '2016-05-01 07:00:00' and '2016-05-07 06:59:59' 
    and e.id = i.shipment_event_id 
) 
group by i.seller_sku; 

Вторая идея: Pre-заполнитель значения

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

select 
    sum(i.pre_sum_quantity), 
    i.seller_sku 
from 
(
    select seller_sku, shipment_event_id, sum(quantity) as pre_sum_quantity 
    from amz_event_shipment_items 
    where exists 
    (
    select * 
    from amz_event_fees f 
    where f.currency = 'USD' 
    and f.shipment_item_id = amz_event_shipment_items.id 
) 
    group by seller_sku, shipment_event_id 
) i 
where exists 
(
    select * 
    from amz_shipment_events e 
    where e.event_type <> 'RefundEvent' 
    and e.posted_date between '2016-05-01 07:00:00' and '2016-05-07 06:59:59' 
    and e.id = i.shipment_event_id 
) 
group by i.seller_sku; 

В случае, если есть только несколько типов событий, вы можете также попытаться избавиться от <>, что делает его немного более вероятно, используется индекс:

where e.event_type in ('EarlyPaymentEvent','LatePaymentEvent') 

(В этом случае он может заплатить, чтобы иметь индекс, где event_type приходит до posted_date.)

Я должен признать, я не думаю, что это будет гораздо быстрее, чем оригинальный EXISTS запрос, но может быть стоит попробовать.

Answer 2

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

select 
    sum(i.quantity), 
    i.seller_sku 
from amz_event_shipment_items i 
where exists 
(
    select * 
    from amz_event_fees f 
    where f.currency = 'USD' 
    and f.shipment_item_id = i.id 
) 
and exists 
(
    select * 
    from amz_shipment_events e 
    where e.event_type <> 'RefundEvent' 
    and e.posted_date between '2016-05-01 07:00:00' and '2016-05-07 06:59:59' 
    and e.id = i.shipment_event_id 
) 
group by i.seller_sku; 

(MySQL, как известно, медленно на IN пунктах иногда, поэтому я использую EXISTS здесь, хотя мне нравится IN лучше.)

Answer 3

Возможно, что один из ваших объединений возвращается больше записей, чем вы ожидаете , Я бы попробовал просто сделать select * и отсортировать по sku и eyeball результаты.