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 2021/09/09 20:11:24 UTC
[GitHub] [accumulo-testing] DomGarguilo opened a new pull request #151: Replace use of the deprecated `newInstance()` method
DomGarguilo opened a new pull request #151:
URL: https://github.com/apache/accumulo-testing/pull/151
While working on #150 I noticed the following while building:
```
[WARNING] /home/dgarguilo/github/accumulo-testing/src/main/java/org/apache/accumulo/testing/scalability/Run.java:[83,9] newInstance() in java.lang.Class has been deprecated
```
It looks like `newInstance()` has been deprecated and replaced by `getDeclaredConstructor().newInstance()`.
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] milleruntime commented on pull request #151: Replace use of the deprecated `newInstance()` method
Posted by GitBox <gi...@apache.org>.
milleruntime commented on pull request #151:
URL: https://github.com/apache/accumulo-testing/pull/151#issuecomment-918140137
> > The Scalability test is pretty old and I don't know of anyone that has run the test in a while. It might be worth it as a follow on to run it and update it. It may be still be a useful test.
>
> I'll go ahead and merge this for now then take a closer look at the scalability test. Was there anything in particular you had in mind in terms of updating it?
Mostly just test to make sure it still works. I took a quick look at the classes and one thing we could do is make a helper script in `<testing-home>/bin` like we do with the other tests. That would make it easier to run at least.
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] milleruntime commented on pull request #151: Replace use of the deprecated `newInstance()` method
Posted by GitBox <gi...@apache.org>.
milleruntime commented on pull request #151:
URL: https://github.com/apache/accumulo-testing/pull/151#issuecomment-916971084
The Scalability test is pretty old and I don't know of anyone that has run the test in a while. It might be worth it as a follow on to run it and update it. It may be still be a useful test.
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] DomGarguilo merged pull request #151: Replace use of the deprecated `newInstance()` method
Posted by GitBox <gi...@apache.org>.
DomGarguilo merged pull request #151:
URL: https://github.com/apache/accumulo-testing/pull/151
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] DomGarguilo commented on pull request #151: Replace use of the deprecated `newInstance()` method
Posted by GitBox <gi...@apache.org>.
DomGarguilo commented on pull request #151:
URL: https://github.com/apache/accumulo-testing/pull/151#issuecomment-917130994
> The Scalability test is pretty old and I don't know of anyone that has run the test in a while. It might be worth it as a follow on to run it and update it. It may be still be a useful test.
I'll go ahead and merge this for now then take a closer look at the scalability test. Was there anything in particular you had in mind in terms of updating it?
--
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: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org