rtrivedi12 commented on PR #6683: URL: https://github.com/apache/hive/pull/6683#issuecomment-6307302751
> @rtrivedi12 - Can you work on fixing the test failures? @saihemanth-cloudera Most of the test failures were related to out file changes after adding authorization. But there are 4 test failures with `Table not found`. Analyzer's earlier version looked up the `LIKE` argument as a literal table name (`getTable(dbName, pattern, true)`). At runtime, though, `ShowTableStatusOperation` treats it as a SQL `LIKE` pattern (`'jsontab%'`, `src_`, `"%"`, `foo_n_`, ...). I have updated analyzer now to resolves the pattern the same way the operation does: it lists the tables in the database, filters them with `UDFLike.likePatternToRegExp`, and adds a `ReadEntity` for each match. So every table whose metadata is returned goes through authorization. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
