asolimando opened a new pull request #2296: URL: https://github.com/apache/calcite/pull/2296
### Aim of the PR improve the feature by relying on the safer fields order from the struct field type itself (i.e., `relType.getFields()[index]`) so that we can enable it by default. ### Organization of the PR The PR is (logically) split into 3 commits to ease the review process: 1. the fix itself 2. refinement/addition of the relevant unit tests 3. removal of the code related to the feature being "experimental" ### How the changes were tested - Relevant unit-tests have been adapted/improved - one unit-test added to cover "null" index case - tests have been validated locally and via CI ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
