[GitHub] [ignite-3] korlov42 commented on a diff in pull request #2181: IGNITE-19709 Sql. Get rid of starting implicit transactions for SELECT statements without FROM keyword, change mapping

[via GitHub]

korlov42 commented on code in PR #2181:
URL: https://github.com/apache/ignite-3/pull/2181#discussion_r1228113886


##########
modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/metadata/IgniteFragmentMapping.java:
##########
@@ -69,13 +75,153 @@ public class IgniteMdFragmentMapping implements 
MetadataHandler<FragmentMappingM
     public static FragmentMapping fragmentMappingForMetadataQuery(RelNode rel, 
RelMetadataQuery mq, MappingQueryContext ctx) {

Review Comment:
   it worth to rename this method



-- 
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: notifications-unsubscr...@ignite.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org