Dmitry Patsura
Dmitry Patsura
@alamb @andygrove is it required to add a new expression for Ballista? I did that in [218a917](https://github.com/apache/arrow-datafusion/pull/2549/commits/218a9177cd625990387ca183995b89bd489c5f5a), but I don't use it. Thanks
Rebased ✅ cC @alamb @tustvold
Right now, DF uses Generic Dialect for `sqlparser`, but it's allowed only for `PostgreSqlDialect` ```rust Token::ShiftLeft if dialect_of!(self is PostgreSqlDialect) => { Some(BinaryOperator::PGBitwiseShiftLeft) } Token::ShiftRight if dialect_of!(self is PostgreSqlDialect) =>...
Looks likes I forget about overflowing here. I think I can use `overflowing_shl` to handle it correctly to the Error https://doc.rust-lang.org/std/primitive.i64.html#method.overflowing_shl ``` ❯ select 2
Panic on overflow was fixed in [474ed5a](https://github.com/apache/arrow-datafusion/pull/3037/commits/474ed5a39d5629147170530dd2b8fa41b02975bd) Without error to be similar with PostgreSQL
Awaiting https://github.com/apache/arrow-datafusion/pull/3072
Marked PR as ready for review ✅
@andygrove > Can we update the user guide to show this new functionality? This can be a follow-on issue. https://github.com/apache/arrow-datafusion/blob/master/docs/source/user-guide/sql/sql_status.md#supported-sql I didn't find any place where I should update the...
@yyaremenko @ddmler Don't forget that We support `"php": ">=5.5"` to run PHPSA
@ddmler Looks like > check minimum php version for analyzers is implemented now I change from [ ] to [X] for it