Skip to content

Rename AnalyzeRequest to AnalyzeTextOptions, and add constructor overloads / helpers for required fields #11830

Description

@heaths

Per discussion,

  • Rename AnalyzeRequest to AnalyzeTextOptions. SearchIndexClient.AnalyzeText should look like: AnalyzeText(string indexName, AnalyzeTextOptions options, ...).
  • Add constructor overloads or other initialization helpers to require text, and either analyer or tokenizer (or, if you want, analyzerName, and tokenizerName, though I'll open up a separate issue for that).
class AnalyzeTextOptions {
  ctor(string text, LexicalAnalyzerName analyzer);
  ctor(string text, LexicalTokenizerName tokenizer);
  string text { get; }
  LexicalAnalyzerName? analyzer { get; }
  LexicalTokenizerName? tokenizer { get; }
  (remaining properties: { get; set; })

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    ClientThis issue points to a problem in the data-plane of the library.Search

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions