Skip to content

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

Closed

Description

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; })
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

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