generated from amazon-archives/__template_Custom
-
Notifications
You must be signed in to change notification settings - Fork 181
Closed
Labels
calcitecalcite migration releatedcalcite migration releatedenhancementNew feature or requestNew feature or request
Description
Is your feature request related to a problem?
The current Calcite remote IT has drawbacks:
- Although enabling fallback by default, Calcite*IT will run most tests without fallback in fact, it has duplication process with NonFallbackCalcite*IT which extends from it. This will increase our IT time cost a lot with unnecessary tests.
- For some tests we originally expect them throw exception(because of incorrect PPL), we don't want to test with fallback to v2 because v2 will throw exception as well. E.g. WhereCommandIT::testInWithIncompatibleType. Testing the error message in separate suite(calcite enable/disable) is enough.
What solution would you like?
Unify Calcite*IT with NonFallbackCalcite*IT together, and override tests with fallback for those we've previous ignored in NonFallbackCalcite*IT
Metadata
Metadata
Assignees
Labels
calcitecalcite migration releatedcalcite migration releatedenhancementNew feature or requestNew feature or request