You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "dlmarion (via GitHub)" <gi...@apache.org> on 2023/12/04 16:19:27 UTC

[PR] Fix disabled test in LargeSplitRowIT [accumulo]

dlmarion opened a new pull request, #4014:
URL: https://github.com/apache/accumulo/pull/4014

   (no comment)


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


Re: [PR] Fix disabled test in LargeSplitRowIT [accumulo]

Posted by "dlmarion (via GitHub)" <gi...@apache.org>.
dlmarion commented on PR #4014:
URL: https://github.com/apache/accumulo/pull/4014#issuecomment-1838996263

   It looks like #3425 may have changed the exception being thrown from AccumuloServerException to AccumuloException. Not sure if that's a big deal or not...


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


Re: [PR] Fix disabled test in LargeSplitRowIT [accumulo]

Posted by "keith-turner (via GitHub)" <gi...@apache.org>.
keith-turner commented on PR #4014:
URL: https://github.com/apache/accumulo/pull/4014#issuecomment-1839028863

   > It looks like https://github.com/apache/accumulo/pull/3425 may have changed the exception being thrown from AccumuloServerException to AccumuloException. Not sure if that's a big deal or not...
   
   I suspect thats caused by split moving from a direct tserver RPC to using the RPCs that run fate operations.  I don't think that change is an issue.


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


Re: [PR] Fix disabled test in LargeSplitRowIT [accumulo]

Posted by "dlmarion (via GitHub)" <gi...@apache.org>.
dlmarion merged PR #4014:
URL: https://github.com/apache/accumulo/pull/4014


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