Postgres novato aqui.
Eu estou querendo saber se esta consulta é otimizada ou não? Tentei JOIN ON apenas os valores que são 100% necessários e deixando todas as condições dinâmicas na cláusula WHERE. Veja abaixo.
SELECT *
FROM
myapp_employees
JOIN myapp_users ON
myapp_users.user_id=myapp_employees.user_id
JOIN myapp_contacts_assoc ON
myapp_contacts_assoc.user_id=myapp_users.user_id
JOIN myapp_contacts ON
myapp_contacts.contact_id=myapp_contacts_assoc.contact_id
WHERE
myapp_contacts.value='[email protected]' AND
myapp_contacts.type=(1)::INT2 AND
myapp_contacts.is_primary=(1)::INT2 AND
myapp_contacts.expired_at IS NULL AND
myapp_employees.status=(1)::INT2 AND
myapp_users.status=(1)::INT2
LIMIT 1;
Nota: Por contexto, este proc está verificando se um usuário também é um funcionário (privs elevados/tipo de usuário diferente).
De qualquer forma, este é o caminho certo a seguir? O JOIN ON deve conter mais instruções como verificar expired_at IS NULL, por exemplo? Por que ou por que isso não faz sentido?
Logicamente , não faz diferença se você coloca condições na cláusula join de an
INNER JOIN
ou naWHERE
cláusula do mesmoSELECT
. O efeito é o mesmo.(Não é o caso de
OUTER JOIN
!)Ao operar com configurações padrão, também não faz diferença para o plano de consulta ou desempenho . O Postgres é livre para reorganizar junções e
JOIN
&WHERE
condições em sua busca pelo melhor plano de consulta - desde que o número de tabelas não seja maior quejoin_collapse_limit
(padrão8
). Detalhes:Para legibilidade e manutenção , faz sentido colocar condições que conectam tabelas na respectiva
JOIN
cláusula e condições gerais naWHERE
cláusula.Sua consulta parece muito bem. No entanto, eu usaria apelidos de tabela para reduzir o ruído.
Detalhe menor:
int2 '1'
ou mesmo1::int2
são mais sensatos do que(1)::INT2
. E ao comparar com um valor de tipo de dados numérico bem definido, uma constante numérica simples1
também é boa o suficiente.Alguns pontos..
Se você estiver participando de uma condição com o mesmo nome (
user_id
) no seu caso, poderá usarUSING (user_id)
em vez deON (a.user_id = b.user_id)
. Isso também evita que uma coluna redundante seja potencialmente gerada (se você estiver executandoSELECT *
em produção).1::int2
é problemático. Oustatus
, eis_primary
e outros já estãoint2
, caso em que o literal 1 será automaticamente convertido em int2, ou int2 convertido em int conforme pg achar adequado. Ou, se você estiver armazenando-os como inteiros regulares e lançando-os como se isso fizesse diferença na computação - o que não faz, o elenco sozinho torna isso uma proposta perdedora.Quando possível, todos os ::int2 provavelmente devem ser armazenados como arquivos
boolean
. Então você pode escrever suaWHERE
condição para ser mais simples também.Para seu tipo e status, você pode querer um
ENUM
tipo.