KiranVelumuri commented on PR #5546: URL: https://github.com/apache/hive/pull/5546#issuecomment-2698254986
Yes, TestHttpSamlAuthentication failed due to the env variables not being available, which help to dynamically decide the correct ClientProviderStrategy I believe. I think it should work, I tried the same but explicitly in a config file and waiting on the results. About the JDBC storage handler, yes, the port 5432 seems occupied by the docker container leading to failure of the test. This failure due to race condition is happening every time in multiple runs, and got fixed when I changed the port 5432 to 5433 for the tests. From your comment, I suppose it should have also failed with 5433 then, but something else is at play which I could not comprehend. -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org
