xueyumusic commented on issue #6445: [FLINK-8302] [table] Add SHIFT_LEFT and 
SHIFT_RIGHT
URL: https://github.com/apache/flink/pull/6445#issuecomment-425967961
 
 
   Thanks for review, @twalthr . I also think that it would be best if calcite 
could support << and >>, however I am not sure whether calcite could support 
this very well for two reason, one is that it looks bitwise shift operation is 
not sql standard and from 
[HIVE-9859](https://issues.apache.org/jira/browse/HIVE-9859) discussion it 
looks that ">>" has potential conflicts with nested complex type declaration in 
hive which also uses calcite.
   So maybe there are two ways:
   1. As you said make calcite suport this.
   2. Since 
[Spark](https://spark.apache.org/docs/2.3.0/api/sql/index.html#shiftleft) and 
[Hive](https://cwiki.apache.org/confluence/display/Hive/LanguageManual+UDF) 
both have shiftleft and shiftright, maybe we could also add this named as 
shiftleft and shiftright...
   
   Thanks

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to