Настройка производительности запроса Oracle SQL - PullRequest
2 голосов
/ 06 апреля 2020

Может кто-нибудь помочь мне настроить этот запрос? Для возврата данных в sqldeveloper требуется 1 минута.

SELECT
  masterid, notification_id, notification_list, typeid,
  subject, created_at, created_by, approver, sequence_no,
  productid, statusid, updated_by, updated_at, product_list,
  notification_status, template, notification_type, classification
FROM 
( 
  SELECT
    masterid, notification_id, notification_list, typeid, subject,
    approver, created_at, created_by, sequence_no, productid,
    statusid, updated_by, updated_at, product_list, notification_status,
    template, notification_type, classification,
    ROW_NUMBER() OVER(ORDER BY masterid DESC)AS r
  FROM 
  (
    SELECT DISTINCT
      a.masterid         AS masterid,
      a.maxid            AS notification_id,
      notification_list,
      typeid,
      noti.subject       AS subject,
      noti.approver      AS approver,
      noti.created_at    AS created_at,
      noti.created_by    AS created_by,
      noti.sequence_no   AS sequence_no,
      a.productid        AS productid,
      a.statusid         AS statusid,
      noti.updated_by    AS updated_by,
      noti.updated_at    AS updated_at,
      (
        SELECT LISTAGG(p.name,',') WITHIN GROUP(ORDER BY p.id) AS list_noti
        FROM product p
        INNER JOIN notification_product np ON np.product_id = p.id
        WHERE  notification_id = a.maxid
      ) AS product_list,
      (
        SELECT description
        FROM notification_status
        WHERE id = a.statusid
      ) AS notification_status,
      (
        SELECT name
        FROM template
        WHERE id = a.templateid
      ) AS template,
      (
        SELECT description
        FROM notification_type
        WHERE id = a.typeid
      ) AS notification_type,
      (
        SELECT tc.description
        FROM template_classification tc
        INNER JOIN notification nt ON tc.id = nt.classification_id
        WHERE  nt.id = a.maxid
      ) AS classification
    FROM
    (
      SELECT
        nm.id                       AS masterid,
        nm.product_id               AS productid,
        nm.notification_status_id   AS statusid,
        nm.template_id              AS templateid,
        nm.notification_type_id     AS typeid,
        (
          SELECT MAX(id)
          FROM notification
          WHERE notification_master_id = nm.id
        ) AS maxid,
        (
          SELECT LISTAGG(n.id,',') WITHIN GROUP(ORDER BY nf.id) AS list_noti
          FROM notification n
          WHERE notification_master_id = nm.id
        ) AS notification_list
      FROM notification_master nm
      INNER JOIN notification nf ON nm.id = nf.notification_master_id
      WHERE nm.disable = 'N'
      ORDER BY nm.id DESC
    ) a
    INNER JOIN notification noti 
      ON a.maxid = noti.id
      AND 
      (
        (
          (
            TO_DATE('01-jan-1970','dd-MM-YYYY') +
            numtodsinterval(created_at / 1000,'SECOND')
           ) < 
           (current_date + INTERVAL '-21' DAY)
        )
        OR (typeid exists(2,4) AND statusid = 4)
      )
  )
)
WHERE r BETWEEN 11 AND 20

1 Ответ

2 голосов
/ 06 апреля 2020

DISTINCT очень часто является индикатором плохо написанного запроса. Нормализованная база данных не содержит дубликатов данных, так откуда же вдруг возникли дубликаты, которые вы должны удалить с помощью DISTINCT? Очень часто это ваш собственный запрос, производящий их. Во-первых, не создавайте дубликаты, так что вам не нужно DISTINCT позже.

В вашем случае вы присоединяетесь к таблице notification в своем подзапросе a, но не используете его строки в этом подзапросе; вы выбираете только из notification_master_id.

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

Некоторые примечания:

  • Чтобы получить описание от template_classification, вы снова присоединяетесь к таблице уведомлений, которая не требуется.
  • ORDER BY в подзапросе (ORDER BY nm.id DESC) является излишним, поскольку результаты подзапроса по стандарту SQL не отсортированы. (Oracle иногда нарушает этот стандарт, чтобы применить ROWNUM к результату, но вы не используете ROWNUM в своем запросе.)
  • Жаль, что вы храните created_at не как DATE или TIMESTAMP, но как число. Это заставляет вас рассчитывать. Я не думаю, что это сильно повлияет на ваш запрос, потому что вы используете его в состоянии OR.
  • CURRENT_DATE дает вам дату клиента. Это редко требуется, так как вы выбираете данные из базы данных, которая, конечно, должна относиться не к какой-то дате клиента, а к собственной дате SYSDATE.

Если я не ошибаюсь, ваш запрос можно сократить до:

SELECT
  nm.id                      AS masterid,
  nf.id                      AS notification_id,
  nfagg.notification_list    AS notification_list,
  nm.notification_type_id    AS typeid,
  nf.subject                 AS subject,
  nf.approver                AS approver,
  nf.created_at              AS created_at,
  nf.created_by              AS created_by,
  nf.sequence_no             AS sequence_no,
  nm.product_id              AS productid,
  nm.notification_status_id  AS statusid,
  nf.updated_by              AS updated_by,
  nf.updated_at              AS updated_at,
  (
    SELECT LISTAGG(p.name, ',') WITHIN GROUP (ORDER BY p.id)
    FROM product p
    INNER JOIN notification_product np ON np.product_id = p.id
    WHERE np.notification_id = nf.id
  ) AS product_list,
  (
    SELECT description
    FROM notification_status
    WHERE id = nm.notification_status_id
  ) AS notification_status,
  (
    SELECT name
    FROM template
    WHERE id = nm.template_id
  ) AS template,
  (
    SELECT description
    FROM notification_type
    WHERE id = nm.notification_type_id
  ) AS notification_type,
  (
    SELECT description
    FROM template_classification
    WHERE id = nf.classification_id
  ) AS classification
FROM notification_master nm
INNER JOIN
(
  SELECT
    notification_master_id,
    MAX(id) AS maxid,
    LISTAGG(id,',') WITHIN GROUP (ORDER BY id) AS notification_list
  FROM notification
  GROUP BY notification_master_id
) nfagg ON nfagg.notification_master_id = nm.id
INNER JOIN notification nf
   ON nf.id = nfagg.maxid
  AND
  (
    (
      DATE '1970-01-01' + NUMTODSINTERVAL(nf.created_at / 1000, 'SECOND')
        < CURRENT_DATE + INTERVAL '-21' DAY
    )
    OR (nm.notification_type_id IN (2,4) AND nm.notification_status_id = 4)
  )
WHERE nm.disable = 'N'
ORDER BY nm.id DESC
OFFSET 10 ROWS
FETCH NEXT 10 ROWS ONLY;

Как уже упоминалось, вы можете заменить CURRENT_DATE на SYSDATE.

Я рекомендую следующие индексы для запроса:

CREATE INDEX idx1 ON notification_master (disable, id, notification_status_id, notification_type_id);
CREATE INDEX idx2 ON notification (notification_master_id, id, created_at);

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

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...