DATA-3163 - Remove all references to TagsByFilter

[tahiyasalam]

Hey folks! Not sure I'm doing this correctly, but we are in the process of deprecating the TagsByFilter API. As part of that, I'd like to remove all references from the SDK before deprecating in proto. LMK if there's another process you all prefer.

[stuqdog]

Heya! the change itself looks reasonable, but I wonder if just removing these methods immediately is the right approach. What is the timeline for removing the TagsByFilter APIs? I think it might make more sense here to leave these methods as they are but add deprecation warnings (see here as an example), that way anyone using these methods will at least have a bit of warning before they go away.

All that said, I have no insight into how much these methods are actually used so I don't feel too strongly! If you prefer to just remove them outright I'm certainly willing to be convinced 🙂

[njooma]

How were the GRPC/protobuf changes made? I think they're still included in the actual proto changes, so we shouldn't remove them here either. if desired, we should only update the client/tests, but we should leave anything in gen or proto dirs as is

[njooma]

Further to @stuqdog's comment, is this API being deprecated or removed? If just deprecated, perhaps we also deprecate them in the SDKs instead of removing them entirely