[GitHub] [accumulo] ctubbsii commented on pull request #3737: Use custom Transport Factory to set Transport message and frame size

["ctubbsii (via GitHub)" <gi...@apache.org>]

ctubbsii commented on PR #3737:
URL: https://github.com/apache/accumulo/pull/3737#issuecomment-1702644095

   Okay, I updated the IT to test for messages that are smaller than the configured amount (which should work), but also included the checks that @dlmarion had to verify it failed if the message was larger than the configured amount. So, we have both now. But, there are a few remaining issues:
   
   * The ssl case does not seem to have the same failure characteristics as the other server types. In some cases, when a message is sent that is larger than what is configured, it still works. I'm not sure why. In other cases, it seemed to not work, like we'd expect, but it would fail with a different message. I had to refactor the test to ensure MiniAccumuloCluster gave it separate directories so this can be debugged further. I don't think it's critical, though. For now, I've ignored the ssl case in the IT.
   * The more interesting thing is that if I reduce the configured max size to a lower amount (say, 20MB), the tests that send a smaller message (even if it is substantially smaller than 20MB) seem to fail with messages in the tserver logs saying that it received a message that was larger. I'm not sure how, since I never sent one that big, and I don't expect Accumulo to natively be sending itself large messages from another server. This one doesn't make sense. I got the IT to pass with 80MB... I haven't found the precise threshold where it stops working, though. Hopefully, having separate directories for the tests now will make it easier to troubleshoot each one separately.


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