morningman commented on issue #67578:
URL: https://github.com/apache/doris/issues/67578#issuecomment-5690860373

   > I would like to take a small, independently testable part of A3: implement 
GetTableTypes and make GetTables honor the complete table_types filter. On 
current master, getStreamTableTypes returns UNIMPLEMENTED, and the helper 
forwards only the first requested type to listTableStatus (which only specially 
handles VIEW). That can omit requested tables or include excluded types.
   > 
   > The proposed patch keeps the existing catalog/user privilege checks, uses 
the same BASE TABLE / SYSTEM VIEW / VIEW names emitted by GetTables, returns 
the type list sorted as Flight SQL specifies, and filters the authorized table 
results before optional schema collection. I am preparing focused FE regression 
tests, including multiple/unknown types and Arrow buffer cleanup. This would be 
a partial A3 contribution, with no session or ADBC-driver redesign. Please let 
me know if someone is already working on this slice or if it should be 
coordinated differently.
   
   Thanks for your contribution. I will review your PR


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

Reply via email to