confluentinc / ksql

The database purpose-built for stream processing applications.
https://ksqldb.io
Other
120 stars 1.04k forks source link

Improved error messages for invalid physical plan upgrades #6087

Open agavra opened 4 years ago

agavra commented 4 years ago

Error messages regarding query upgrades when the physical plans can be cryptic to the user as they refer to internal concepts such as:

- StreamAggregate must have matching nonAggregateColumns. Values differ: [`ID`, `ROWTIME`] vs. [`ID`, `COL1`] 
- Upgrades not yet supported for TableSuppress

We should come up with a better way to make these understandable, while still keeping the actionable information.

vcrfxia commented 3 years ago

A number of users have expressed confusion about this regarding why ksqlDB doesn't support query upgrades for table aggregations or queries on windowed or joined sources, and requested we add a short explanation for why into the error message (in addition to making it less cryptic).