You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@hive.apache.org by "vihangk1 (via GitHub)" <gi...@apache.org> on 2023/03/31 12:04:30 UTC
[GitHub] [hive] vihangk1 opened a new pull request, #4179: HIVE-27202: Disable flaky test testComplexQuery
vihangk1 opened a new pull request, #4179:
URL: https://github.com/apache/hive/pull/4179
### What changes were proposed in this pull request?
`TestJdbcWithMiniLlapRow#testComplexQuery` is flaky and fails intermittently on branch-3. Here is the flaky test run http://ci.hive.apache.org/job/hive-flaky-check/634/
### Why are the changes needed?
branch-3 stability and green runs.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Tested locally that the test is skipped.
--
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-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] vihangk1 commented on pull request #4179: HIVE-27202: Disable flaky test testComplexQuery
Posted by "vihangk1 (via GitHub)" <gi...@apache.org>.
vihangk1 commented on PR #4179:
URL: https://github.com/apache/hive/pull/4179#issuecomment-1492394248
> @vihangk1 Can we increase the timeout in this test. Will that work? I agree this is flaky for sure.
I didn't want to get into fixing the test just yet but because it was a simple thing I am going to try running the flaky test job using an increased timeout of 120000. If it passes, I will update the PR with the increased timeout.
--
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-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] vihangk1 merged pull request #4179: HIVE-27202: Disable flaky test testComplexQuery
Posted by "vihangk1 (via GitHub)" <gi...@apache.org>.
vihangk1 merged PR #4179:
URL: https://github.com/apache/hive/pull/4179
--
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-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] amanraj2520 commented on pull request #4179: HIVE-27202: Disable flaky test testComplexQuery
Posted by "amanraj2520 (via GitHub)" <gi...@apache.org>.
amanraj2520 commented on PR #4179:
URL: https://github.com/apache/hive/pull/4179#issuecomment-1491872764
@vihangk1 Can we increase the timeout in this test. Will that work? I agree this is flaky for sure.
--
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-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] vihangk1 commented on pull request #4179: HIVE-27202: Disable flaky test testComplexQuery
Posted by "vihangk1 (via GitHub)" <gi...@apache.org>.
vihangk1 commented on PR #4179:
URL: https://github.com/apache/hive/pull/4179#issuecomment-1491826785
@amanraj2520 Would you like to review this? I found that this test fails on unrelated changes.
--
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-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] amanraj2520 commented on pull request #4179: HIVE-27202: Disable flaky test testComplexQuery
Posted by "amanraj2520 (via GitHub)" <gi...@apache.org>.
amanraj2520 commented on PR #4179:
URL: https://github.com/apache/hive/pull/4179#issuecomment-1493221476
+1 them, LGTM
--
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-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] vihangk1 commented on pull request #4179: HIVE-27202: Disable flaky test testComplexQuery
Posted by "vihangk1 (via GitHub)" <gi...@apache.org>.
vihangk1 commented on PR #4179:
URL: https://github.com/apache/hive/pull/4179#issuecomment-1493218820
@amanraj2520 I increased the timeout to 120s to test if this removes the flakiness of this test. It improves the stability of the test by a little bit but still remains flaky due to some other issue. I think we should disable this test.
http://ci.hive.apache.org/job/hive-flaky-check/637/
--
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-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] vihangk1 commented on pull request #4179: HIVE-27202: Disable flaky test testComplexQuery
Posted by "vihangk1 (via GitHub)" <gi...@apache.org>.
vihangk1 commented on PR #4179:
URL: https://github.com/apache/hive/pull/4179#issuecomment-1493530822
Merging this without a green run because this is for the branch-3 stability. `testCliDriver[tez_input_counters] – org.apache.hadoop.hive.cli.split20.TestMiniLlapLocalCliDriver` seems flaky as well and we should disable it in a separate 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: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org