calcite icon indicating copy to clipboard operation
calcite copied to clipboard

Published 20 hours ago • verified publisher icon apache

[CALCITE-4754] Fixed wrong SQL generated for PIVOT queries on Redshift.

Open javierRivas opened this issue 3 years ago • 1 comments

This PR fixes the SQL produced by Calcite for a PIVOT query targeting Redshift

javierRivas avatar Aug 27 '21 01:08 javierRivas

The changes LGTM, however, I find a bit weird that IS TRUE is mentioned all over the code but not featuring at all in the unit tests. Can you expand the test cases to cover it explicitly or elaborate on why you think it's not needed?

asolimando avatar Sep 11 '21 15:09 asolimando