You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Then the SqlOpStringSerializerImpl incorrectly returns null to indicate a query with no result rows.
The solution is to properly keep track of whether aggregations are being used in a query, and if so, serialize the empty table as e.g. SELECT NULL WHERE FALSE.
The text was updated successfully, but these errors were encountered:
Added a somewhat hacky fix to SqlOpSelectBlock's isEmpty() method:
The method now checks if the projection contains a single argument that is a count function.
However, the isEmpty() method for SqlOpProjection and SqlOpExtend are currently still broken - i.e. they would incorrectly return true.
If we have
Then the SqlOpStringSerializerImpl incorrectly returns null to indicate a query with no result rows.
The solution is to properly keep track of whether aggregations are being used in a query, and if so, serialize the empty table as e.g.
SELECT NULL WHERE FALSE
.The text was updated successfully, but these errors were encountered: