Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: batch insert length assertion #2042

Merged
merged 3 commits into from
Aug 5, 2024
Merged

fix: batch insert length assertion #2042

merged 3 commits into from
Aug 5, 2024

Conversation

rafaelcr
Copy link
Collaborator

@rafaelcr rafaelcr commented Aug 5, 2024

Fixes #2037

@rafaelcr rafaelcr requested a review from zone117x August 5, 2024 16:18
zone117x
zone117x approved these changes Aug 5, 2024
View reviewed changes
Copy link
Member

@zone117x zone117x left a comment

Choose a reason for hiding this comment

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

💯

@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 5, 2024 •
edited
Loading

Vercel deployment URL: https://stacks-blockchain-3uu3poktc-hirosystems.vercel.app 🚀

@codecov Codecov
Copy link

codecov bot commented Aug 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@rafaelcr rafaelcr merged commit fe720d0 into master Aug 5, 2024
29 checks passed
@rafaelcr rafaelcr deleted the fix/batch-assert branch August 5, 2024 16:52
blockstack-devops pushed a commit that referenced this pull request Aug 5, 2024
@semantic-release-bot
chore(release): 7.13.2 [skip ci]
f6e50f6 
## [7.13.2](v7.13.1...v7.13.2) (2024-08-05)

### Bug Fixes

* batch insert length assertion ([#2042](#2042)) ([fe720d0](fe720d0))
@blockstack-devops
Copy link

🎉 This PR is included in version 7.13.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@smcclellan smcclellan linked an issue Aug 5, 2024 that may be closed by this pull request
Code has an Assertion Error that causes the API to exit abnormally #2037
Closed
zone117x pushed a commit that referenced this pull request Aug 8, 2024
@rafaelcr @zone117x
fix: batch insert length assertion (#2042)
994640f 
* fix: batch insert length assertion

* build: upgrade docker-compose

* build: use docker compose
zone117x added a commit that referenced this pull request Aug 13, 2024
@zone117x @rafaelcr
feat!: refactor from Express to Fastify
365060a 
* feat: route splitting between express and fastify

* feat: refactor `/extended/v1/tx/*` endpoints to fastify

* feat: refactor `/extended/v1/stx_supply/*` endpoints to fastify

* feat: refactor `/extended/v1/info/*` endpoints to fastify

* feat: refactor `/extended/v1/tokens/*` endpoints to fastify

* feat: refactor `/extended/v1/tokens/*` endpoints to fastify

* feat: refactor `/extended/v1/contract/*` endpoints to fastify

* feat: refactor `/extended/v1/fee_rate/*` endpoints to fastify

* feat: refactor `/extended/v1/microblock/*` endpoints to fastify

* feat: refactor `/extended/v1/block/*` endpoints to fastify

* feat: refactor `/extended/v1/burnchain/*` endpoints to fastify

* feat: refactor `/extended/v1/address/*` endpoints to fastify

* feat: refactor `/extended/v1/search/*` endpoints to fastify

* feat: refactor `/extended/v1/pox*` endpoints to fastify

* feat: refactor `/extended/v1/faucets/*` endpoints to fastify

* feat: refactor `/extended/v1/debug/*` endpoints to fastify

* feat: refactor `/extended/v2/blocks/*` and `/extended/v2/burn-blocks/*` endpoints to fastify

* feat: refactor `/extended/v2/smart-contracts/*` endpoints to fastify

* feat: refactor `/extended/v2/mempool/*` endpoints to fastify

* feat: refactor `/extended/v2/pox/*` endpoints to fastify

* feat: refactor `/extended/v2/addresses/*` endpoints to fastify

* feat: refactor `/v1/names/*` endpoints to fastify

* feat: refactor `/v1/namespaces/*` endpoints to fastify

* feat: refactor `/v1/addresses/*` endpoints to fastify

* feat: refactor `/v2/prices/*` endpoints to fastify

* feat: refactor core-node RPC proxy (/v2/*) to fastify

* chore: remove dead code

* feat: openAPI spec generation from fastify routes

* chore: remove references to legacy generated types

* docs: missing openapi tag on burn-blocks route

* docs: update docs and client generation scripts

* fix: several query params should be optional

* fix: only use a GET route for extended status

* chore: simpify typing for TransactionSchema and MempoolTransactionSchema

* feat: refactor client library

* chore: remove dependencies on old generated types

* chore: isolate rosetta json schemas and delete the rest

* chore: cleanup prometheus metrics

* fix: misc tests

* test: misc rosetta fixes

* fix: batch insert length assertion (#2042)

* fix: batch insert length assertion

* build: upgrade docker-compose

* build: use docker compose

* test: misc bns test fixes

* test: misc pox fixes

* ci: misc fixes

* chore: fix unused exports lint

* chore: simplify docs and client package.json scripts

* feat: refactor event-server from express to fastify

* chore: expose more helper types in the client lib

---------

Co-authored-by: Rafael Cárdenas <rafael@rafaelcr.com>
blockstack-devops pushed a commit that referenced this pull request Aug 13, 2024
@semantic-release-bot
chore(release): 8.0.0-beta.1 [skip ci]
9f78825 
## [8.0.0-beta.1](v7.13.2...v8.0.0-beta.1) (2024-08-13)

### ⚠ BREAKING CHANGES

* refactor from Express to Fastify

### Features

* refactor from Express to Fastify ([365060a](365060a)), closes [#2042](#2042)
zone117x added a commit that referenced this pull request Aug 13, 2024
@zone117x @rafaelcr
feat!: refactor from Express to Fastify
aa0e51e 
* feat: route splitting between express and fastify

* feat: refactor `/extended/v1/tx/*` endpoints to fastify

* feat: refactor `/extended/v1/stx_supply/*` endpoints to fastify

* feat: refactor `/extended/v1/info/*` endpoints to fastify

* feat: refactor `/extended/v1/tokens/*` endpoints to fastify

* feat: refactor `/extended/v1/tokens/*` endpoints to fastify

* feat: refactor `/extended/v1/contract/*` endpoints to fastify

* feat: refactor `/extended/v1/fee_rate/*` endpoints to fastify

* feat: refactor `/extended/v1/microblock/*` endpoints to fastify

* feat: refactor `/extended/v1/block/*` endpoints to fastify

* feat: refactor `/extended/v1/burnchain/*` endpoints to fastify

* feat: refactor `/extended/v1/address/*` endpoints to fastify

* feat: refactor `/extended/v1/search/*` endpoints to fastify

* feat: refactor `/extended/v1/pox*` endpoints to fastify

* feat: refactor `/extended/v1/faucets/*` endpoints to fastify

* feat: refactor `/extended/v1/debug/*` endpoints to fastify

* feat: refactor `/extended/v2/blocks/*` and `/extended/v2/burn-blocks/*` endpoints to fastify

* feat: refactor `/extended/v2/smart-contracts/*` endpoints to fastify

* feat: refactor `/extended/v2/mempool/*` endpoints to fastify

* feat: refactor `/extended/v2/pox/*` endpoints to fastify

* feat: refactor `/extended/v2/addresses/*` endpoints to fastify

* feat: refactor `/v1/names/*` endpoints to fastify

* feat: refactor `/v1/namespaces/*` endpoints to fastify

* feat: refactor `/v1/addresses/*` endpoints to fastify

* feat: refactor `/v2/prices/*` endpoints to fastify

* feat: refactor core-node RPC proxy (/v2/*) to fastify

* chore: remove dead code

* feat: openAPI spec generation from fastify routes

* chore: remove references to legacy generated types

* docs: missing openapi tag on burn-blocks route

* docs: update docs and client generation scripts

* fix: several query params should be optional

* fix: only use a GET route for extended status

* chore: simpify typing for TransactionSchema and MempoolTransactionSchema

* feat: refactor client library

* chore: remove dependencies on old generated types

* chore: isolate rosetta json schemas and delete the rest

* chore: cleanup prometheus metrics

* fix: misc tests

* test: misc rosetta fixes

* fix: batch insert length assertion (#2042)

* fix: batch insert length assertion

* build: upgrade docker-compose

* build: use docker compose

* test: misc bns test fixes

* test: misc pox fixes

* ci: misc fixes

* chore: fix unused exports lint

* chore: simplify docs and client package.json scripts

* feat: refactor event-server from express to fastify

* chore: expose more helper types in the client lib

---------

Co-authored-by: Rafael Cárdenas <rafael@rafaelcr.com>
blockstack-devops pushed a commit that referenced this pull request Aug 13, 2024
@semantic-release-bot
chore(release): 8.0.0-beta.1 [skip ci]
183ddd0 
## [8.0.0-beta.1](v7.13.2...v8.0.0-beta.1) (2024-08-13)

### ⚠ BREAKING CHANGES

* refactor from Express to Fastify

### Features

* refactor from Express to Fastify ([aa0e51e](aa0e51e)), closes [#2042](#2042)
zone117x added a commit that referenced this pull request Aug 15, 2024
@zone117x
Merge branch 'refactor/fastify-wip' into chore/v8.0.0-beta.1
4bc8f77 
* refactor/fastify-wip: (56 commits)
  docs: note RPC client library change
  fix: tx event pagination limit
  docs: client library migration guide
  fix: ensure height-or-hash params are parsed correctly and have correct openapi and client types
  ci: fix lint and tests
  chore: expose more helper types to client lib
  fix: openapi and client support for comma-separated query params
  ci: fix client npm lib building
  chore: expose more helper types in the client lib
  feat: refactor event-server from express to fastify
  chore: simplify docs and client package.json scripts
  chore: fix unused exports lint
  ci: misc fixes
  test: misc pox fixes
  test: misc bns test fixes
  fix: batch insert length assertion (#2042)
  test: misc rosetta fixes
  fix: misc tests
  chore: cleanup prometheus metrics
  chore: isolate rosetta json schemas and delete the rest
  ...

# Conflicts:
#	client/src/generated/schema.d.ts
#	client/src/index.ts
#	client/src/types.d.ts
#	docs/openapi.json
#	docs/openapi.yaml
#	src/api/routes/tx.ts
#	src/api/routes/v2/blocks.ts
#	src/api/routes/v2/burn-blocks.ts
#	src/api/routes/v2/schemas.ts
#	src/api/schemas/entities/transaction-events.ts
#	src/api/schemas/entities/transactions.ts
#	src/api/schemas/params.ts
#	src/api/schemas/util.ts
#	src/datastore/pg-store-v2.ts
zone117x added a commit that referenced this pull request Aug 15, 2024
@zone117x @rafaelcr
fix: pagination and query param parsing bugs
a382d2b 
* feat: route splitting between express and fastify

* feat: refactor `/extended/v1/tx/*` endpoints to fastify

* feat: refactor `/extended/v1/stx_supply/*` endpoints to fastify

* feat: refactor `/extended/v1/info/*` endpoints to fastify

* feat: refactor `/extended/v1/tokens/*` endpoints to fastify

* feat: refactor `/extended/v1/tokens/*` endpoints to fastify

* feat: refactor `/extended/v1/contract/*` endpoints to fastify

* feat: refactor `/extended/v1/fee_rate/*` endpoints to fastify

* feat: refactor `/extended/v1/microblock/*` endpoints to fastify

* feat: refactor `/extended/v1/block/*` endpoints to fastify

* feat: refactor `/extended/v1/burnchain/*` endpoints to fastify

* feat: refactor `/extended/v1/address/*` endpoints to fastify

* feat: refactor `/extended/v1/search/*` endpoints to fastify

* feat: refactor `/extended/v1/pox*` endpoints to fastify

* feat: refactor `/extended/v1/faucets/*` endpoints to fastify

* feat: refactor `/extended/v1/debug/*` endpoints to fastify

* feat: refactor `/extended/v2/blocks/*` and `/extended/v2/burn-blocks/*` endpoints to fastify

* feat: refactor `/extended/v2/smart-contracts/*` endpoints to fastify

* feat: refactor `/extended/v2/mempool/*` endpoints to fastify

* feat: refactor `/extended/v2/pox/*` endpoints to fastify

* feat: refactor `/extended/v2/addresses/*` endpoints to fastify

* feat: refactor `/v1/names/*` endpoints to fastify

* feat: refactor `/v1/namespaces/*` endpoints to fastify

* feat: refactor `/v1/addresses/*` endpoints to fastify

* feat: refactor `/v2/prices/*` endpoints to fastify

* feat: refactor core-node RPC proxy (/v2/*) to fastify

* chore: remove dead code

* feat: openAPI spec generation from fastify routes

* chore: remove references to legacy generated types

* docs: missing openapi tag on burn-blocks route

* docs: update docs and client generation scripts

* fix: several query params should be optional

* fix: only use a GET route for extended status

* chore: simpify typing for TransactionSchema and MempoolTransactionSchema

* feat: refactor client library

* chore: remove dependencies on old generated types

* chore: isolate rosetta json schemas and delete the rest

* chore: cleanup prometheus metrics

* fix: misc tests

* test: misc rosetta fixes

* fix: batch insert length assertion (#2042)

* fix: batch insert length assertion

* build: upgrade docker-compose

* build: use docker compose

* test: misc bns test fixes

* test: misc pox fixes

* ci: misc fixes

* chore: fix unused exports lint

* chore: simplify docs and client package.json scripts

* feat: refactor event-server from express to fastify

* chore: expose more helper types in the client lib

* ci: fix client npm lib building

* fix: openapi and client support for comma-separated query params

* chore: expose more helper types to client lib

* ci: fix lint and tests

* fix: ensure height-or-hash params are parsed correctly and have correct openapi and client types

* docs: client library migration guide

* fix: tx event pagination limit

* docs: note RPC client library change

---------

Co-authored-by: Rafael Cárdenas <rafael@rafaelcr.com>
blockstack-devops pushed a commit that referenced this pull request Aug 15, 2024
@semantic-release-bot
chore(release): 8.0.0-beta.2 [skip ci]
9dee0c0 
## [8.0.0-beta.2](v8.0.0-beta.1...v8.0.0-beta.2) (2024-08-15)

### Bug Fixes

* pagination and query param parsing bugs ([a382d2b](a382d2b)), closes [#2042](#2042)
zone117x added a commit that referenced this pull request Aug 19, 2024
@zone117x @rafaelcr
feat!: refactor from express to fastify (#2045)
bd65fcf 
* feat: route splitting between express and fastify

* feat: refactor `/extended/v1/tx/*` endpoints to fastify

* feat: refactor `/extended/v1/stx_supply/*` endpoints to fastify

* feat: refactor `/extended/v1/info/*` endpoints to fastify

* feat: refactor `/extended/v1/tokens/*` endpoints to fastify

* feat: refactor `/extended/v1/tokens/*` endpoints to fastify

* feat: refactor `/extended/v1/contract/*` endpoints to fastify

* feat: refactor `/extended/v1/fee_rate/*` endpoints to fastify

* feat: refactor `/extended/v1/microblock/*` endpoints to fastify

* feat: refactor `/extended/v1/block/*` endpoints to fastify

* feat: refactor `/extended/v1/burnchain/*` endpoints to fastify

* feat: refactor `/extended/v1/address/*` endpoints to fastify

* feat: refactor `/extended/v1/search/*` endpoints to fastify

* feat: refactor `/extended/v1/pox*` endpoints to fastify

* feat: refactor `/extended/v1/faucets/*` endpoints to fastify

* feat: refactor `/extended/v1/debug/*` endpoints to fastify

* feat: refactor `/extended/v2/blocks/*` and `/extended/v2/burn-blocks/*` endpoints to fastify

* feat: refactor `/extended/v2/smart-contracts/*` endpoints to fastify

* feat: refactor `/extended/v2/mempool/*` endpoints to fastify

* feat: refactor `/extended/v2/pox/*` endpoints to fastify

* feat: refactor `/extended/v2/addresses/*` endpoints to fastify

* feat: refactor `/v1/names/*` endpoints to fastify

* feat: refactor `/v1/namespaces/*` endpoints to fastify

* feat: refactor `/v1/addresses/*` endpoints to fastify

* feat: refactor `/v2/prices/*` endpoints to fastify

* feat: refactor core-node RPC proxy (/v2/*) to fastify

* chore: remove dead code

* feat: openAPI spec generation from fastify routes

* chore: remove references to legacy generated types

* docs: missing openapi tag on burn-blocks route

* docs: update docs and client generation scripts

* fix: several query params should be optional

* fix: only use a GET route for extended status

* chore: simpify typing for TransactionSchema and MempoolTransactionSchema

* feat: refactor client library

* chore: remove dependencies on old generated types

* chore: isolate rosetta json schemas and delete the rest

* chore: cleanup prometheus metrics

* fix: misc tests

* test: misc rosetta fixes

* fix: batch insert length assertion (#2042)

* fix: batch insert length assertion

* build: upgrade docker-compose

* build: use docker compose

* test: misc bns test fixes

* test: misc pox fixes

* ci: misc fixes

* chore: fix unused exports lint

* chore: simplify docs and client package.json scripts

* feat: refactor event-server from express to fastify

* chore: expose more helper types in the client lib

* ci: fix client npm lib building

* fix: openapi and client support for comma-separated query params

* chore: expose more helper types to client lib

* ci: fix lint and tests

* fix: ensure height-or-hash params are parsed correctly and have correct openapi and client types

* docs: client library migration guide

* fix: tx event pagination limit

* docs: note RPC client library change

* fix: tx event-limit default should be 100

* fix: missing event limit max overrides on a few endpoints

* chore: use PrincipalSchema in multiple areas

* chore: use UntilBlockSchema in multiple areas

* fix: perform status endpoint sql inside transactions

* chore: do not commit openapi schema files

* chore: fix tests

---------

Co-authored-by: Rafael Cárdenas <rafael@rafaelcr.com>
blockstack-devops pushed a commit that referenced this pull request Aug 27, 2024
@semantic-release-bot
chore(release): 8.0.0-beta.6 [skip ci]
be5a889 
## [8.0.0-beta.6](v8.0.0-beta.5...v8.0.0-beta.6) (2024-08-27)

### ⚠ BREAKING CHANGES

* refactor from express to fastify (#2045)

### Features

* cursor-based pagination on blocks endpoint ([#2060](#2060)) ([bfdcce1](bfdcce1))
* export events tsv directly to postgres instance ([#2048](#2048)) ([f401a0f](f401a0f))
* export events tsv directly to postgres instance ([#2048](#2048)) ([#2058](#2058)) ([a1f5b12](a1f5b12))
* refactor from express to fastify ([#2045](#2045)) ([bd65fcf](bd65fcf)), closes [#2042](#2042)

### Bug Fixes

* index on `principal_stx_txs` table for faster `/v1/address/{addr}/transactions` lookups ([#2059](#2059)) ([ab64ab7](ab64ab7))
blockstack-devops pushed a commit that referenced this pull request Aug 28, 2024
@semantic-release-bot
chore(release): 8.0.0 [skip ci]
a38a35f 
## [8.0.0](v7.14.1...v8.0.0) (2024-08-28)

### ⚠ BREAKING CHANGES

* refactor from express to fastify (#2045)
* refactor from Express to Fastify

### Features

* cursor-based pagination on blocks endpoint ([#2060](#2060)) ([bfdcce1](bfdcce1))
* export events tsv directly to postgres instance ([#2048](#2048)) ([f401a0f](f401a0f))
* refactor from Express to Fastify ([aa0e51e](aa0e51e)), closes [#2042](#2042)
* refactor from express to fastify ([#2045](#2045)) ([bd65fcf](bd65fcf)), closes [#2042](#2042)

### Bug Fixes

* missing event limit max overrides on a few endpoints ([4f70930](4f70930))
* pagination and query param parsing bugs ([a382d2b](a382d2b)), closes [#2042](#2042)
* perform status endpoint sql inside transactions ([b23445c](b23445c))
* tx event-limit default should be 100 ([32d0670](32d0670))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zone117x zone117x zone117x approved these changes

Assignees
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Code has an Assertion Error that causes the API to exit abnormally
3 participants
@rafaelcr @blockstack-devops @zone117x