tigrisdata-archive / tigris-client-ts

TypeScript client for Tigris
https://www.tigrisdata.com/docs/sdkstools/typescript/
Apache License 2.0
15 stars 10 forks source link

fix: Wrap all RPC and Http requests behind a driver #379

Closed garrensmith closed 9 months ago

garrensmith commented 1 year ago

What type of PR is this? (check all applicable)

Description

This is the start of supporting HTTP and grpc in the typescript client. All grpc calls are now wrapped behind a grpc driver with a common interface that the HTTP can implement as well.

Related Tickets & Documents

Added/updated tests?

Is this change backwards compatible?

Does it require updates to Tigris docs?

Checklist

[optional] Are there any post deployment tasks we need to perform?

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 88.62% and project coverage change: +2.74 :tada:

Comparison is base (a527ec4) 90.37% compared to head (58c219c) 93.12%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #379 +/- ## ========================================== + Coverage 90.37% 93.12% +2.74% ========================================== Files 29 26 -3 Lines 5644 4448 -1196 Branches 662 477 -185 ========================================== - Hits 5101 4142 -959 + Misses 542 304 -238 - Partials 1 2 +1 ``` | [Impacted Files](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/379?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata) | Coverage Δ | | |---|---|---| | [src/tigris.ts](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/379?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata#diff-c3JjL3RpZ3Jpcy50cw==) | `70.98% <46.66%> (+1.71%)` | :arrow_up: | | [src/cache.ts](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/379?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata#diff-c3JjL2NhY2hlLnRz) | `84.82% <54.54%> (+31.74%)` | :arrow_up: | | [src/search/search-index.ts](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/379?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata#diff-c3JjL3NlYXJjaC9zZWFyY2gtaW5kZXgudHM=) | `83.33% <66.66%> (+9.94%)` | :arrow_up: | | [src/tokensupplier.ts](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/379?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata#diff-c3JjL3Rva2Vuc3VwcGxpZXIudHM=) | `95.12% <95.12%> (ø)` | | | [src/collection.ts](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/379?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata#diff-c3JjL2NvbGxlY3Rpb24udHM=) | `98.11% <96.49%> (+1.55%)` | :arrow_up: | | [src/db.ts](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/379?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata#diff-c3JjL2RiLnRz) | `94.06% <100.00%> (+0.08%)` | :arrow_up: | | [src/index.ts](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/379?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata#diff-c3JjL2luZGV4LnRz) | `100.00% <100.00%> (ø)` | | | [src/search/index.ts](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/379?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata#diff-c3JjL3NlYXJjaC9pbmRleC50cw==) | `100.00% <100.00%> (ø)` | | | [src/search/result.ts](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/379?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata#diff-c3JjL3NlYXJjaC9yZXN1bHQudHM=) | `100.00% <100.00%> (+0.51%)` | :arrow_up: | | [src/search/search.ts](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/379?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata#diff-c3JjL3NlYXJjaC9zZWFyY2gudHM=) | `100.00% <100.00%> (+2.40%)` | :arrow_up: | | ... and [3 more](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/379?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata) | | ... and [5 files with indirect coverage changes](https://app.codecov.io/gh/tigrisdata/tigris-client-ts/pull/379/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tigrisdata)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.