xuzifu666 commented on code in PR #4620:
URL: https://github.com/apache/calcite/pull/4620#discussion_r2516528773
##########
core/src/test/resources/org/apache/calcite/test/SqlHintsConverterTest.xml:
##########
@@ -373,7 +371,6 @@ TableScan:[[PROPERTIES inheritPath:[] options:{K1=v1,
K2=v2}], [PROPERTIES inher
<Resource name="hints">
<![CDATA[
Project:[[FAST_SNAPSHOT inheritPath:[] options:[PRODUCTS_TEMPORAL]]]
-Snapshot:[[FAST_SNAPSHOT inheritPath:[0, 1] options:[PRODUCTS_TEMPORAL]]]
Review Comment:
@dssysolyatin Thank you very much for your guidance. I didn't understand
this part deeply enough before, but I have a deeper understanding with this
opportunity. According to the suggestion, all the current problems have been
addressed.
The point you have considered is worth thinking about, but historical
adaptation is currently solved in this way. As far as solving this jira is
concerned, this PR should be fine.
Maybe a new Jira can be opened next to improve the problem you raised. It
involved should be large.
--
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]
