[GitHub] [ignite] alex-plekhanov commented on a change in pull request #9176: IGNITE-13549 CREATE INDEX/DROP INDEX commands

[GitBox <gi...@apache.org>]

alex-plekhanov commented on a change in pull request #9176:
URL: https://github.com/apache/ignite/pull/9176#discussion_r655330939



##########
File path: modules/calcite/src/main/codegen/includes/parserImpls.ftl
##########
@@ -164,6 +164,77 @@ SqlCreate SqlCreateTable(Span s, boolean replace) :
     }
 }
 
+SqlNode IndexedColumn() :
+{
+    final Span s;
+    SqlNode col;
+}
+{
+    col = SimpleIdentifier()
+    (
+        <ASC>
+    |   <DESC> {

Review comment:
       Currently, it's not supported by the Ignite-H2 engine as well and not supported by core indexes. As a first goal, I think we should provide the same functionality as the Ignite-H2 engine, perhaps later NULL's ordering can be added. I can create another ticket if you think this functionality will be helpful.




-- 
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:
users@infra.apache.org