Uh oh!
There was an error while loading. Please reload this page.
HBASE-22677 Add unit tests for org.apache.hadoop.hbase.util.ByteRangeUtils and Classes - #374
Conversation
Apache-HBase
commented
Jul 12, 2019
💔 -1 overall
This message was automatically generated. |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
7dc8d6d to
6f0496fCompareApache-HBase
commented
Jul 15, 2019
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 16, 2019
🎊 +1 overall
This message was automatically generated. |
Braavos6
commented
Jul 16, 2019
Hi @HorizonNet Is there any other changes you would like me to make to this PR? |
Apache-HBase
commented
Jul 17, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 20, 2019
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 22, 2019
🎊 +1 overall
This message was automatically generated. |
Braavos6
commented
Jul 22, 2019
Hi @HorizonNet just following up Is there any other changes you would like me to make to this PR? |
Apache-HBase
commented
Jul 22, 2019
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 22, 2019
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 22, 2019
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 22, 2019
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 23, 2019
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 23, 2019
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 23, 2019
🎊 +1 overall
This message was automatically generated. |
HorizonNet
left a comment
There was a problem hiding this comment.
Left a minor NIT. Remaining stuff LGTM.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
6f0496f to
6bd6e08CompareBraavos6
commented
Jul 23, 2019
Hi @HorizonNet I have replaced the imports with static imports as requested :) |
Apache-HBase
commented
Jul 23, 2019
💔 -1 overall
This message was automatically generated. |
HorizonNet
commented
Jul 23, 2019
@Braavos96 There are two Checkstyle warnings related to the import order. Could you please resolve them and then we should be ready to merge. |
6bd6e08 to
d4989bfCompareBraavos6
commented
Jul 23, 2019
Hi @HorizonNet I have changed the order of the static assert imports |
Apache-HBase
commented
Jul 23, 2019
💔 -1 overall
This message was automatically generated. |
HorizonNet
commented
Jul 23, 2019
@Braavos96 Imports are still in the wrong order. If I remember correctly they should come first before any other imports. You can also check that by running |
….Utils and Classes These tests were written using Diffblue Cover.
d4989bf to
c2fe8ffCompareBraavos6
commented
Jul 23, 2019
Hi @HorizonNet I moved the static imports to the top. I also tried the command |
Apache-HBase
commented
Jul 29, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 29, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 30, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 30, 2019
💔 -1 overall
This message was automatically generated. |
Braavos6
commented
Jul 30, 2019
Hi @HorizonNet is there any other changes to be made to this PR? |
HorizonNet
commented
Jul 30, 2019
@Braavos96 Yes, the build is still failing (see above). |
Apache-HBase
commented
Jul 30, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 30, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 30, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 30, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 30, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 31, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 31, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 31, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Jul 31, 2019
💔 -1 overall
This message was automatically generated. |
xcangCRM
commented
Jul 31, 2019
+1 |
Apache-HBase
commented
Jul 31, 2019
💔 -1 overall
This message was automatically generated. |
xcangCRM
commented
Jul 31, 2019
@Braavos96 |
HorizonNet
commented
Jul 31, 2019
@xcangCRM I'm doing it, as soon as the last Checkstyle issue is resolved (see my comments and build output above). |
Apache-HBase
commented
Aug 1, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Aug 1, 2019
💔 -1 overall
This message was automatically generated. |
Apache9
commented
Aug 1, 2019
Please finish this @HorizonNet ? Thanks. |
Apache-HBase
commented
Aug 1, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Aug 1, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Aug 2, 2019
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Aug 4, 2019
💔 -1 overall
This message was automatically generated. |
HorizonNet
commented
Aug 4, 2019
@Apache9 I added a commit, which should resolve the last Checkstyle issue. Let's wait for QA. |
Apache-HBase
commented
Aug 4, 2019
🎊 +1 overall
This message was automatically generated. |
I've analysed your codebase and noticed that
org.apache.hadoop.hbase.util.ByteRangeUtilsandClassesis not fully tested.I've written some tests for the methods in this class with the help of Diffblue Cover.
Hopefully, these tests will help you detect any regressions caused by future code changes. If you would find it useful to have additional tests written for this repository, I would be more than happy to look at other classes that you consider important in a subsequent PR.