[release-24.0] Preserve query hints in field/impossible queries (#20366)

[vitess-bot]

Description

This is a backport of #20366

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[vitess-bot]

Hello @arthurschreiber, there are conflicts in this backport.

Please address them in order to merge this Pull Request. You can execute the snippet below to reset your branch and resolve the conflict manually.

Make sure you replace origin by the name of the vitessio/vitess remote

git fetch --all
gh pr checkout 20399
git reset --hard origin/release-24.0
git cherry-pick -m 1 72a42a5c57c48ae06548d5014f181ede6b148d4c

[arthurschreiber]

Resolved the committed merge conflict in go/vt/sqlparser/impossible_query.go.

The conflict arose because release-24.0's FormatImpossibleQuery does not carry the from dual refactor that exists on main. The upstream PR (#20366) only intends to preserve query hints in the field/impossible query, so I applied just that change — adding node.Comments to the select output — on top of release-24.0's existing structure, leaving the from handling untouched.

Unit tests pass: TestFormatImpossibleQuery (sqlparser) and the full planbuilder testdata suite.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.