DomGarguilo commented on PR #5398: URL: https://github.com/apache/accumulo/pull/5398#issuecomment-2773030732
> I kinda get lost when I look at this w/ all the TInfo changes. Are there any classes in particular that you want reviewed besides AccumuloProtocol? Yea there are a lot of changed files from getting rid of the TInfo object. The other changes here are mostly limited to AccumuloProtocolFactory, ThriftUtil and TServerUtils and even then there is not a whole lot that has changed outside of AccumuloProtocolFactory. -- 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-unsubscr...@accumulo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org