rahil-c commented on code in PR #18347:
URL: https://github.com/apache/hudi/pull/18347#discussion_r2965532352
##########
hudi-spark-datasource/hudi-spark3.3.x/src/main/scala/org/apache/spark/sql/parser/HoodieSpark3_3ExtendedSqlParser.scala:
##########
@@ -128,7 +128,8 @@ class HoodieSpark3_3ExtendedSqlParser(session:
SparkSession, delegate: ParserInt
normalized.contains("create index") ||
normalized.contains("drop index") ||
normalized.contains("show indexes") ||
- normalized.contains("refresh index")
+ normalized.contains("refresh index") ||
+ normalized.contains(" blob")
Review Comment:
same concern as expressed here
https://github.com/apache/hudi/pull/18098#discussion_r2956370028 (although
understand now this is not related to the read_blob).
However trying to understand if this would match for things not related to
creating a blob col, for example:
```
-- A table with a column named blob_path — NOT a BLOB type column
CREATE TABLE t (id BIGINT, blob_path STRING)
```
If this is not a real concern let me know just wanted to bring it up.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
