Manish

Results 73 comments of Manish

> Also address [#21502 (comment)](https://github.com/trinodb/trino/pull/21502#issuecomment-2067027490) to extend TestHiveMetastoreMetadataQueriesAccessOperations please sure, will update for this one.

> > Also address [#21502 (comment)](https://github.com/trinodb/trino/pull/21502#issuecomment-2067027490) to extend TestHiveMetastoreMetadataQueriesAccessOperations please > > sure, will update for this one. @anusudarsan ref: https://github.com/trinodb/trino/pull/21502#discussion_r1575427996 we have now a Smoke Test cases and `TestHiveMetastoreCatalogs`...

hi @dain if you get time, can you please review it. took care of the feedback from @anusudarsan @findinpath thanks to @anusudarsan @findinpath for continuous feedback and support!

sorry, got engaged in other work, will try to take care of remaining comments in next a few days.

> @osscm I pushed some changes instead of you. Thanks a lot @ebyhr!

> @osscm Why did you revert my change? did I revert? Apologies! maybe by mistake while squashing in local.

> @osscm I restored to the status you reverted my change. Please add a fixup commit for additional changes. > @osscm I restored to the status you reverted my change....

> @osscm the CI looks good now @anusudarsan, so good to merge now? :) @ebyhr or @dain if you can please help to merge, thanks!

if the metadata is cached then also this would be a problem?

Sure, Pardon for the delay, I’m away for few days, will do this weekend. Thanks! On Mon, Jul 17, 2023 at 4:50 PM Manfred Moser ***@***.***> wrote: > Could you...