forked from facebookincubator/velox
-
Notifications
You must be signed in to change notification settings - Fork 3
Open
Labels
bugSomething isn't workingSomething isn't working
Description
Bug description
https://github.com/gibber9809/velox/blob/timestamp-marshalling/velox/type/parser/TypeParser.ll
It seems the parser will drop the special chars like $ or \.
After changing WORD ([[:alpha:][:alnum:]_]*) to WORD ([[:alpha:][:alnum:]_\$\\]*), the issue is addressed.
But I don't think we can modify this in OSS PR.
System information
Velox System Info v0.0.2
Commit: fa11ccc
CMake Version: 3.28.3
System: Linux-4.15.0-197-generic
Arch: x86_64
C++ Compiler: /usr/bin/c++
C++ Compiler Version: 11.4.0
C Compiler: /usr/bin/cc
C Compiler Version: 11.4.0
CMake Prefix Path: /usr/local;/usr;/;/usr/local/lib/python3.10/dist-packages/cmake/data;/usr/local;/usr/X11R6;/usr/pkg;/opt
\nThe results will be copied to your clipboard if xclip is installed.
Relevant logs
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working