-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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.
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
Updating to the newest version of pg-types #2547
Comments
@brianc What do you think? |
I think there are some backwards incompatible changes but I'm fine to do a semver bump and up the version of pg types. I'll start looking into this. |
I just noticed that we would need the latest version of pg-types to work with |
@sgronblo You can always override the |
Are there any plans on how to proceed? The old For now, I could work around this using npm overrides to force "overrides": {
"pg": {
"pg-types": "^4.0.1"
}
}, |
I'd love to see this happen to pick up the new version of |
the old postgres-date also have some serious performance issues which was mitigated in bendrucker/postgres-date#21, any plan to continue work on this? |
@tleiroblox Was there some kind of pathological date case that I missed? 3x speedup on parsing doesn’t sound like “serious performance issues”. |
For time series data that has date in every row, parseDate old implementation alone could easily cost more than 50% of the cpu cycles pg used, which often become the bottleneck of the whole node application. It's also not memory efficient that can result in more gc (which also lead to cpu usage) |
Currently, the pg library uses a very old version of pg-types.
Is there something, in particular, that is stopping us from updating to the newest version of the pg-types library?
The text was updated successfully, but these errors were encountered: