Support the string concat operator #178
Conversation
Support the string concat operator
src/tokenizer.rs
Outdated
| // a regular '/' operator | ||
| _ => Err(TokenizerError(format!( | ||
| "Tokenizer Error at Line: {}, Col: {}", |
There was a problem hiding this comment.
The reference to '/' is presumably a typo. This should probably say we don't support the single-pipe | token (bitwise OR in some dialects, but not part of the standard)
There was a problem hiding this comment.
Made some modifications to the error message.
Probably, it would be useful though to support | too.
There was a problem hiding this comment.
Also, another improvement would be to include a simple function which adds line/column information in a consistent way.
There was a problem hiding this comment.
Probably, it would be useful though to support
|too.
Yes, I see no reason not to.
include a simple function which adds line/column information in a consistent way.
Agreed. I see you filed #179 about that.
|
Thanks! |
Adds support for string concatenation operator
||https://github.com/andygrove/sqlparser-rs/issues/160