You're mixing two functions. You could use either:

TOWNSHIP || ',' || RANGE || ',' || ISP_SECTIO

or

coalesce( TOWNSHIP, ',', RANGE, ',', ISP_SECTIO )

The coalesce version is a bit safer, since it will automatically treat NULL values like an empty string.

• Note use of "coalesce", not "concatenate" (which is a different function)

You're mixing two functions. You could use either:

TOWNSHIP || ',' || RANGE || ',' || ISP_SECTIO

or

concat( TOWNSHIP, ',', RANGE, ',', ISP_SECTIO )

The concat version is a bit safer, since it will automatically treat NULL values like an empty string.

• Note use of "concat", not "concatenate" (which is a different function)