krconv commented on PR #7665: URL: https://github.com/apache/hbase/pull/7665#issuecomment-3820461721
What do you think of also deprecating `AsyncTable#getRequestAttributes()`, now that the behavior isn't well defined (i.e. the return value could change based on which thread it is called from)? I like your suggestion to merge the two request attribute setters on AsyncTableBuilder into one, but the complication I see is that now getRequestAttributes() no longer has any static set of attributes to return. I'm not sure why `getRequestAttributes()` would be useful to anyone, but nevertheless it might be harder for a user to migrate away from using that. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
