You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2020/02/11 15:47:56 UTC
[GitHub] [accumulo] MuazmaZ opened a new pull request #1504: Updated
AccumuloITBase.java to enforce a default timeout
MuazmaZ opened a new pull request #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504
Updated default TimeoutSeconds() method to be 10 minutes
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] keith-turner commented on issue #1504: Updated
AccumuloITBase.java to enforce a default timeout
Posted by GitBox <gi...@apache.org>.
keith-turner commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504#issuecomment-587504173
I am currently looking into these test failures. Rerunning the failed test on this branch and in master.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] keith-turner commented on issue #1504: Updated
AccumuloITBase.java to enforce a default timeout
Posted by GitBox <gi...@apache.org>.
keith-turner commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504#issuecomment-586544236
Saw the following failures
```
[ERROR] Failures:
[ERROR] UnorderedWorkAssignerReplicationIT.dataWasReplicatedToThePeer:288 Drain did not finish within 30 seconds
[ERROR] Errors:
[ERROR] org.apache.accumulo.test.MetaRecoveryIT.test(org.apache.accumulo.test.MetaRecoveryIT)
[ERROR] Run 1: MetaRecoveryIT.test:97 » TestTimedOut test timed out after 240 seconds
[ERROR] Run 2: MetaRecoveryIT.test » Appears to be stuck in thread Time-limited test-SendThr...
[INFO]
[ERROR] org.apache.accumulo.test.functional.MetadataMaxFilesIT.test(org.apache.accumulo.test.functional.MetadataMaxFilesIT)
[ERROR] Run 1: MetadataMaxFilesIT.test:80 » TestTimedOut test timed out after 300 seconds
[ERROR] Run 2: MetadataMaxFilesIT.test » Appears to be stuck in thread Time-limited test-Sen...
[INFO]
[ERROR] org.apache.accumulo.test.replication.UnorderedWorkAssignerReplicationIT.dataWasReplicatedToThePeerWithoutDrain(org.apache.accumulo.test.replication.UnorderedWorkAssignerReplicationIT)
[ERROR] Run 1: UnorderedWorkAssignerReplicationIT.dataWasReplicatedToThePeerWithoutDrain:601 » TestTimedOut
[ERROR] Run 2: UnorderedWorkAssignerReplicationIT.dataWasReplicatedToThePeerWithoutDrain » A...
[INFO]
[INFO]
[ERROR] Tests run: 610, Failures: 1, Errors: 3, Skipped: 1
```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] MuazmaZ commented on issue #1504: Updated
AccumuloITBase.java to enforce a default timeout
Posted by GitBox <gi...@apache.org>.
MuazmaZ commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504#issuecomment-584719269
@ctubbsii I didn't see any test, in particular, to be problematic
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] keith-turner commented on issue #1504: Updated
AccumuloITBase.java to enforce a default timeout
Posted by GitBox <gi...@apache.org>.
keith-turner commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504#issuecomment-586468568
I can run the ITs for this branch.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] keith-turner merged pull request #1504: Updated
AccumuloITBase.java to enforce a default timeout
Posted by GitBox <gi...@apache.org>.
keith-turner merged pull request #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] ctubbsii commented on issue #1504: Updated
AccumuloITBase.java to enforce a default timeout
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504#issuecomment-584709467
@MuazmaZ Were you able to run the ITs with this timeout? Did you notice any particularly problematic tests that might need to be tweaked as a result of this change?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] keith-turner commented on issue #1504: Updated
AccumuloITBase.java to enforce a default timeout
Posted by GitBox <gi...@apache.org>.
keith-turner commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504#issuecomment-587564513
I ran MetadataMaxFilesIT and MetaRecoveryIT a few times in this branch and the master branch. For both branches the test would fail sometimes and succeed sometimes. So those test seem to be a bit flaky even w/o this change.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] ctubbsii commented on issue #1504: Updated
AccumuloITBase.java to enforce a default timeout
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504#issuecomment-584740191
I would be happy to merge this in now, but I'm seeing other failures currently on the master branch, and would like to see those resolved first.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] MuazmaZ commented on issue #1504: Updated
AccumuloITBase.java to enforce a default timeout
Posted by GitBox <gi...@apache.org>.
MuazmaZ commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504#issuecomment-587595634
Thanks @keith-turner @ctubbsii and @billierinaldi for resolving this. I also ran maven verify and got no errors.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] billierinaldi commented on issue #1504: Updated
AccumuloITBase.java to enforce a default timeout
Posted by GitBox <gi...@apache.org>.
billierinaldi commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504#issuecomment-584725797
I'm attempting mvn verify with this change on my laptop now (with a timeout.factor). But I haven't run a verify on this laptop before, so I don't know whether this would typically succeed.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] ctubbsii commented on issue #1504: Updated
AccumuloITBase.java to enforce a default timeout
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504#issuecomment-586489036
> I can run the ITs for this branch.
@keith-turner Can you clarify whether you mean:
1. You *will* run the ITs on this branch to see if this is okay to merge, or
2. You *have* run the ITs on this branch and you think it's okay to merge now.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] keith-turner commented on issue #1504: Updated
AccumuloITBase.java to enforce a default timeout
Posted by GitBox <gi...@apache.org>.
keith-turner commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504#issuecomment-587571394
@MuazmaZ thanks for the contribution. If you would like to add yourself as a contributor to the Accumulo people page [please submit a PR](https://github.com/apache/accumulo-website/edit/master/pages/people.md).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [accumulo] keith-turner commented on issue #1504: Updated
AccumuloITBase.java to enforce a default timeout
Posted by GitBox <gi...@apache.org>.
keith-turner commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout
URL: https://github.com/apache/accumulo/pull/1504#issuecomment-586490960
@ctubbsii I have the ITs running now for this branch merged into master. If it all is well I can merge.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services