Skip to content

Conversation

@philkra
Copy link
Contributor

@philkra philkra commented Mar 26, 2021

as titled.

@delvedor, I wanted to validated the request as well but ran into a type casting issue:

expectAssignable<T.WatcherStatsRequest>({
  "metric": "all",
  "emit_stacktraces": "true"
})

and emit_stacktraces is of type boolean in WatcherStatsRequest.ts 🤷

Copy link
Member

@delvedor delvedor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very likely the yaml test is using "true" instead of true. It should be fixed there.

@delvedor delvedor merged commit bfd0f24 into master Mar 29, 2021
@delvedor delvedor deleted the watcher.stats branch March 29, 2021 07:53
l-trotta added a commit that referenced this pull request Feb 8, 2024
l-trotta added a commit that referenced this pull request Feb 19, 2024
l-trotta added a commit that referenced this pull request Feb 19, 2024
* added 'remove_binary' to AttachmentProcessor (issue #727)

* making node info path optional (issue #242)

* making node info discover seed_host optional (issue #242)

* changed description for update by query  (issue #315)

* adding custom options to index settings similarity (issue #299)

* initial_master_nodes from string to string[] (issue #242)

* correct contrib

* correct node info discover class (issue #242)
l-trotta added a commit that referenced this pull request Feb 19, 2024
l-trotta added a commit that referenced this pull request Feb 19, 2024
l-trotta added a commit that referenced this pull request Feb 19, 2024
* added 'remove_binary' to AttachmentProcessor (issue #727)

* making node info path optional (issue #242)

* making node info discover seed_host optional (issue #242)

* changed description for update by query  (issue #315)

* initial_master_nodes from string to string[] (issue #242)

* correct node info discover class (issue #242)

* generate and contrib
l-trotta added a commit that referenced this pull request Feb 19, 2024
* added 'remove_binary' to AttachmentProcessor (issue #727)

* making node info path optional (issue #242)

* making node info discover seed_host optional (issue #242)

* changed description for update by query  (issue #315)

* adding custom options to index settings similarity (issue #299)

* initial_master_nodes from string to string[] (issue #242)

* correct contrib

* correct node info discover class (issue #242)
l-trotta added a commit that referenced this pull request Feb 19, 2024
* Batch from java issues (#2412)

* added 'remove_binary' to AttachmentProcessor (issue #727)

* making node info path optional (issue #242)

* making node info discover seed_host optional (issue #242)

* changed description for update by query  (issue #315)

* adding custom options to index settings similarity (issue #299)

* initial_master_nodes from string to string[] (issue #242)

* correct contrib

* correct node info discover class (issue #242)

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants