Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BEAM-7794] DynamoDBIOTest is blocking forever #9131

Closed
wants to merge 1 commit into from

Conversation

iemejia
Copy link
Member

@iemejia iemejia commented Jul 23, 2019

This just disables the blocking tests mentioned in the ML

R: @akedin
CC: @cmachgodaddy

@iemejia iemejia requested a review from akedin July 23, 2019 19:55
@iemejia
Copy link
Member Author

iemejia commented Jul 23, 2019

Anton can you check this works in your machine, we disabled only the tests but it may be that the issue is in the @BeforeClass / @AfterClass methods, if that's the case I will do a PR to disable the full test.

@akedin
Copy link
Contributor

akedin commented Jul 23, 2019

Will check in a second, thank you for taking a look

@akedin
Copy link
Contributor

akedin commented Jul 23, 2019

Ignored these tests on my machine, it still hangs with the same symptoms. Ignored the whole file - passes, so it does look like it comes from @Before/After/Rule.

Unrelated to this change, I also see :sdks:java:io:amazon-web-services:spotbugsMain failing to connect to a gradle process (all other spotbugsMains seems to have finished), then times out and fails the build.

@iemejia
Copy link
Member Author

iemejia commented Jul 24, 2019

Arrghh I accidentally pushed to the release branch the updated version of the cherry pick of this PR. Sorry.
Can you help me confirm it does not get stucked anymore @akedin. Thanks

@iemejia iemejia closed this Jul 24, 2019
@akedin
Copy link
Contributor

akedin commented Jul 24, 2019

Works on release branch for me now. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants