presto icon indicating copy to clipboard operation
presto copied to clipboard

Handle Nullpointer when schema name is null

Open agrawalreetika opened this issue 2 years ago • 3 comments

Fixes #18039

agrawalreetika avatar Jul 19 '22 06:07 agrawalreetika

Can you please look in to the build failures?

jainxrohit avatar Jul 19 '22 17:07 jainxrohit

Hi @jainxrohit, All checks are green, please have a look.

agrawalreetika avatar Jul 20 '22 07:07 agrawalreetika

LGTM % a comment. Did we find out why the schema is coming as null in the Postgres use case?

I haven't dug deeper, but I noticed in Metadata calls for listing columns schema and tables names are empty for any catalog when the system catalog is being queried.

agrawalreetika avatar Aug 04 '22 15:08 agrawalreetika

@ajaygeorge Is it good to merge?

agrawalreetika avatar Aug 12 '22 06:08 agrawalreetika

@agrawalreetika yup. LGTM. Already approved.

ajaygeorge avatar Aug 17 '22 22:08 ajaygeorge

@ajaygeorge Can you merge this? Or someone else has to do it?

agrawalreetika avatar Aug 18 '22 03:08 agrawalreetika

@ajaygeorge Can you merge this? Or someone else has to do it?

Adding committer, @tdcmeehan can you please take a look.

ajaygeorge avatar Aug 18 '22 17:08 ajaygeorge