Jackie-Jiang commented on PR #10139:
URL: https://github.com/apache/pinot/pull/10139#issuecomment-1397807868

   Whitespaces in table name is definitely not recommended, but I don't think 
we reject it right now. Actually it is also not recommended to have '-' in 
table name or column name because you have to quote the name in the query, or 
it won't compile because '-' will be parsed as `minus`.
   I'd suggest changing the regex to `(.*?)_(OFFLINE|REALTIME)` which supports 
all possible table names, but also limit the table type value to prevent 
unexpected behavior.


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to