[source-ashby] Paginators inject `per_page`, but the Ashby API documents `limit`
team/useconnectors/source/source-ashby
### What
Every paginated stream in `source-ashby`'s manifest injects page size as `per_page`, but the Ashby API does not have a `per_page` parameter. Its `.list` request schemas are `additionalProperties: false` and document the page size field as `limit` (e.g. `ApplicationListRequest`: `syncToken`, `cursor`, `limit`, `createdAfter`, `createdBefore`, `status`, `jobId`, `expand`).
The likely effect is that the configured page size is silently ignored on every stream and Ashby falls back to its own default page size, inflating the request count and depressing throughput. This connector's measured sustained throughput is roughly 1.31 requests/sec single-threaded, and page size being ignored is a plausible contributor.
Not yet confirmed against a live account — it is inferred from the documented request schemas, since no Ashby credentials are available in GSM or 1Password. Confirming it takes one authenticated request comparing the returned page length with the requested page size.
### Why it is filed separately
Correcting `per_page` to `limit` changes the request shape and the page count for every stream in the connector, so it wants its own version bump and its own regression run rather than riding along inside an unrelated change. Surfaced while adding the `application_history` stream in https://github.com/airbytehq/airbyte/pull/84392, which uses the documented `limit` field for the new stream only and deliberately leaves the existing streams untouched.
Note that https://github.com/airbytehq/airbyte/pull/84274 is also open against this manifest.
---
**Internal Tracking:** https://github.com/airbytehq/oncall/issues/13305
2 条评论