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/06/07 15:15:57 UTC

[GitHub] [accumulo] milleruntime opened a new issue #2147: Revaluate Sunny Test profile

milleruntime opened a new issue #2147:
URL: https://github.com/apache/accumulo/issues/2147


   The sunny test profile feel a little incomplete as it is currently only running 4 ITs (@Category(SunnyDayTests.class):
   <pre>
   test/src/main/java/org/apache/accumulo/test/functional/NativeMapIT.java:@Category(SunnyDayTests.class)
   test/src/main/java/org/apache/accumulo/test/rpc/ThriftBehaviorIT.java:@Category(SunnyDayTests.class)
   test/src/main/java/org/apache/accumulo/test/InMemoryMapIT.java:@Category(SunnyDayTests.class)
   test/src/main/java/org/apache/accumulo/test/start/KeywordStartIT.java:@Category(SunnyDayTests.class)
   </pre>
   
   I think we could add a few ITs to make sure we have basic functionality covered. I think including ITs dealing with compactions, the metadata table, bulk import or write ahead logs would be good.


-- 
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



[GitHub] [accumulo] milleruntime closed issue #2147: Revaluate Sunny Test profile

Posted by GitBox <gi...@apache.org>.
milleruntime closed issue #2147:
URL: https://github.com/apache/accumulo/issues/2147


   


-- 
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] milleruntime commented on issue #2147: Revaluate Sunny Test profile

Posted by GitBox <gi...@apache.org>.
milleruntime commented on issue #2147:
URL: https://github.com/apache/accumulo/issues/2147#issuecomment-912582316


   I missed some when I opened this ticket. There are actually 7 Sunny day ITs. The 4 above plus these:
   <pre>
   main/java/org/apache/accumulo/test/ShellIT.java:@Category({MiniClusterOnlyTests.class, SunnyDayTests.class})
   main/java/org/apache/accumulo/test/functional/ReadWriteIT.java:@Category({StandaloneCapableClusterTests.class, SunnyDayTests.class})
   main/java/org/apache/accumulo/test/ShellServerIT.java:@Category({MiniClusterOnlyTests.class, SunnyDayTests.class})
   </pre>
   
   This is a lot better than what I initially thought. Maybe a bulk import and WAL test would be nice.


-- 
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] ctubbsii commented on issue #2147: Revaluate Sunny Test profile

Posted by GitBox <gi...@apache.org>.
ctubbsii commented on issue #2147:
URL: https://github.com/apache/accumulo/issues/2147#issuecomment-856092440


   As long as it's limited to basic functionality. The idea is that these tests should be very limited, and be a very quick sanity check to make sure that core functionality isn't broken. They should be fast to run, highly reliable and never flaky, and be able to run on meager hardware.


-- 
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