MIME-Version: 1.0
References: <72d71614-3fa0-1a27-c271-a83ee759941a@appl-ecosys.com> <1971486.1767890927@sss.pgh.pa.us>
In-Reply-To: <1971486.1767890927@sss.pgh.pa.us>
From: Ron Johnson <ronljohnsonjr@gmail.com>
Date: Thu, 8 Jan 2026 12:03:29 -0500
Message-ID: <CANzqJaByeiwyRBzfZ1Dqc8A4ZbuTA3nRD9nrF0sWNizk=U6pUQ@mail.gmail.com>
Subject: Re: Not seeing script error
To: pgsql-general <pgsql-general@postgresql.org>
Content-Type: multipart/alternative; boundary="0000000000000068370647e36418"
Archived-At: <https://www.postgresql.org/message-id/CANzqJaByeiwyRBzfZ1Dqc8A4ZbuTA3nRD9nrF0sWNizk%3DU6pUQ%40mail.gmail.com>
Precedence: bulk

--0000000000000068370647e36418
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Thu, Jan 8, 2026 at 11:48=E2=80=AFAM Tom Lane <tgl@sss.pgh.pa.us> wrote:

> Rich Shepard <rshepard@appl-ecosys.com> writes:
> > The script:
> > select p.person_nbr, p.fname, p.lname, p.job_title, p.direct_phone,
> p.email,
> >         c.company_nbr, c.company_name, c.industry
> > from people as p
> > inner join companies as c on p.company_nbr =3D c.company_nbr
> > where p.email is not null and
> > industry =3D 'Chemicals' or
> > industry =3D 'Energy' or
> > industry =3D 'Food processor' or
> > industry =3D 'Manufacturing' or
> > industry =3D 'Maritime' or
> > industry =3D 'Transportation' or
> > industry =3D 'Wood products'
> > group by p.person_nbr, c.company_nbr
> > order by p.person_nbr;
>
> > The where condition, `p.email is not null' is not working; the results
> > include rows where email is null while all other columns are okay.
>
> > I had that condition following the industry conditions but that makes n=
o
> > difference.
>
> > What have I missed?
>
> AND binds more tightly than OR.  I think you meant
>
> where p.email is not null and
> (industry =3D 'Chemicals' or
> industry =3D 'Energy' or
> industry =3D 'Food processor' or
> industry =3D 'Manufacturing' or
> industry =3D 'Maritime' or
> industry =3D 'Transportation' or
> industry =3D 'Wood products')
>
> BTW, using IN might make this more compact.
>

While missing parentheses would not break the logic.

--=20
Death to <Redacted>, and butter sauce.
Don't boil me, I'm still alive.
<Redacted> lobster!

--0000000000000068370647e36418
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr">On Thu, Jan 8, 2026 at 11:48=E2=80=AFAM T=
om Lane &lt;<a href=3D"mailto:tgl@sss.pgh.pa.us">tgl@sss.pgh.pa.us</a>&gt; =
wrote:</div><div class=3D"gmail_quote gmail_quote_container"><blockquote cl=
ass=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid=
 rgb(204,204,204);padding-left:1ex">Rich Shepard &lt;<a href=3D"mailto:rshe=
pard@appl-ecosys.com" target=3D"_blank">rshepard@appl-ecosys.com</a>&gt; wr=
ites:<br>
&gt; The script:<br>
&gt; select p.person_nbr, p.fname, p.lname, p.job_title, p.direct_phone, p.=
email,<br>
&gt;=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0c.company_nbr, c.company_name, c.indu=
stry<br>
&gt; from people as p<br>
&gt; inner join companies as c on p.company_nbr =3D c.company_nbr<br>
&gt; where p.email is not null and<br>
&gt; industry =3D &#39;Chemicals&#39; or<br>
&gt; industry =3D &#39;Energy&#39; or<br>
&gt; industry =3D &#39;Food processor&#39; or<br>
&gt; industry =3D &#39;Manufacturing&#39; or<br>
&gt; industry =3D &#39;Maritime&#39; or<br>
&gt; industry =3D &#39;Transportation&#39; or<br>
&gt; industry =3D &#39;Wood products&#39;<br>
&gt; group by p.person_nbr, c.company_nbr<br>
&gt; order by p.person_nbr;<br>
<br>
&gt; The where condition, `p.email is not null&#39; is not working; the res=
ults<br>
&gt; include rows where email is null while all other columns are okay.<br>
<br>
&gt; I had that condition following the industry conditions but that makes =
no<br>
&gt; difference.<br>
<br>
&gt; What have I missed?<br>
<br>
AND binds more tightly than OR.=C2=A0 I think you meant<br>
<br>
where p.email is not null and<br>
(industry =3D &#39;Chemicals&#39; or<br>
industry =3D &#39;Energy&#39; or<br>
industry =3D &#39;Food processor&#39; or<br>
industry =3D &#39;Manufacturing&#39; or<br>
industry =3D &#39;Maritime&#39; or<br>
industry =3D &#39;Transportation&#39; or<br>
industry =3D &#39;Wood products&#39;)<br>
<br>
BTW, using IN might make this more compact.<br></blockquote><div><br></div>=
<div>While missing=C2=A0parentheses would not break the logic.</div><div><b=
r></div></div><span class=3D"gmail_signature_prefix">-- </span><br><div dir=
=3D"ltr" class=3D"gmail_signature"><div dir=3D"ltr">Death to &lt;Redacted&g=
t;, and butter sauce.<div>Don&#39;t boil me, I&#39;m still alive.<br><div><=
div>&lt;Redacted&gt; lobster!</div></div></div></div></div></div>

--0000000000000068370647e36418--