* Updated code generation
* Switched request and response generics position
* Updated test
* API generation
* Removed unused generics
* Test type definitions for callback style API as well
* Fix comments
* Fix conflict
* API generation
* Updated type def
* Updated abort behavior
- Support for aborting a request with the promise api
- Aborting a request will cause a RequestAbortedError
- Normalized Connection class errors, now every error returned is
wrapped by the client errors constructors
* Updated test
* Updated docs
* Updated code generation script
* Renamed test
* Code coverage
* Avoid calling twice transport.request
* Updated types generation script
* Refactored api method definitions
* Updated test
- Removed old test code
- Added tsd dev dependency
- Rewritten test with tsd
* Removed unused dependencies
* Fixed definition
* Updated test
* Updated docs
* Improved events type definitions
* Updated test
* Minor fixes in the type definitons
* More type test
* Improved Transport type definitions
* Updated test
* Addressed comments
* Code generation
* Use RequestBody, Response and Context everywhere, also default Context to unknown
* Updated test
* body -> hasBody
* Fixed conflicts
* Updated code generation
* Improved request body type definition
* Updated code generation
* Use BodyType for both request and reponses generics
- Use extends for defining the RequestBody generic to force the user
following the same shape.
- BodyType and NDBodyType now accepts a generics to allow injecting
more specific types in the future
* API generation
* Updated test
* Updated docs
* Use BodyType also in ReponseError
* Removed useless client generics
* Renamed generics and types
- prefixed all generics with a T
- BodyType => RequestBody
- NDBodyType => RequestNDBody
- Added ResponseBody
* Updated test
* Updated docs
* Test ResponseBody as well
* Simplify overloads
* API generation
* Updated test
* Updated error types
In very few cases, some API uses the same key for both url and query params, such as the bulk method.
The client is not designed to handle such cases since accepts both url and query keys in the same object, and the url parameter will always take precedence.
This pr fixes this edge case by adding a `querystring` key in the options object.
Fixes: https://github.com/elastic/elasticsearch-js/pull/778
```js
client.bulk({
index: 'index',
type: '_doc',
body: [...]
}, {
querystring: {
type: '_doc'
}
}, console.log)
```
With this pr we introduce the `client.child` API, which returns a new client instance that shares the connection pool with the parent client.
This feature can be handy if you need to have multiple client instances with different configurations, but with a shared connection pool.
Example:
```js
const { Client } = require('@elastic/elasticsearch')
const client = new Client({ node: 'http://localhost:9200' })
const child = client.child({
headers: { 'x-foo': 'bar' },
requestTimeout: 1000
})
client.info(console.log)
child.info(console.log)
```
**Open questions:**
* Currently, the event emitter is shared between the parent and the child(ren), is it ok?
* Currently, if you extend the parent client, the child client will have the same extensions, while if the child client adds an extension, the parent client will not be extended. Is it ok?
**Caveats:**
* You can override _any_ option except for the connection pool specific options (`ssl`, `agent`, `pingTimeout`, `Connection`, and `resurrectStrategy`).
* You can't specify a new `Connection` class.
* If you call `close` in any of the parent/child clients, every client will be closed.
_Note: the `nodeFilter` and `nodeSelector` options are now `Transport` options and no longer `ConnectionPool` options._
