yashmayya opened a new pull request, #13467: URL: https://github.com/apache/pinot/pull/13467
- Last year, many v1 integration tests were configured to also run on the v2 multi-stage query engine in https://github.com/apache/pinot/pull/11404. Many tests were marked to be skipped due to either differences in behavior between v1 and v2 or unsupported features in v2. - Over time, many gaps have been plugged and most integration tests should now be able to run on both the query engines. While some patches updated ITs as well (https://github.com/apache/pinot/pull/13255), not all did, and this patch enables more v1 integration tests to run on the v2 multi-stage query engine. - In some cases, we need to add some additional branches to tests in order to make them work with v2. For instance: - Use of `arrayToMV` for MV columns in filters / aggregations (see https://github.com/apache/pinot/pull/11117/). - Using `X''` for byte literal strings instead of simply `''`. - Explicit casts for timestamp fields being treated as a `LONG` / `BIGINT`. - Slight differences in error codes returned by the two query engines in different query scenarios. - Skip small subsets of tests that use functionality not yet present in v2 like support for `BIG_DECIMAL` and expression overrides (https://github.com/apache/pinot/pull/8319). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
