Refactor SearchServiceClient into SearchIndexClient and SearchIndexerClient

[heaths]

To avoid too many methods in SearchServiceClient, we will refactor it into two separate clients that group the functionality into index management, and skillsets (enrichments) management. Method names will retain their names they have today, like CreateIndex, DeleteIndex, CreateDataSource, DeleteDataSource, etc. This also means there should be the GetServiceStatistics like we should have no, instead of dropping "Service". This differentiates it with GetIndexStatistics.

• SearchIndexClient will aggregate:

  • Index operations: CreateIndex, etc.
  • Synonym map operations: CreateSynonymMap, etc.
  • Service operations: GetServiceStatistics
  • Define/retain GetSearchClient that takes a string indexName and returns a SearchClient

• SearchIndexerClient will aggregate:

  • Indexer operations: CreateIndexer, etc.
  • Data source operations: CreateDataSource, etc.
  • Skillset operations: CreateSkillset, etc.

[bryevdv]

@xiangyan99 Do you have a preference about #11342 ? I have split that PR in to very fine-grained commits it might be simpler for you to cherry pick the ones you want to keep, given this and other new rename changes in the original rename issue. Alternatively we can merge #11342 now as-is if that's easier to start from. What do you think?

[xiangyan99]

Either one works.

[xiangyan99]

Fixed in #11572