You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "domoritz (via GitHub)" <gi...@apache.org> on 2023/06/11 19:31:00 UTC
[GitHub] [arrow] domoritz opened a new pull request, #36034: GH-36033: [JS] Remove BigInt compat
domoritz opened a new pull request, #36034:
URL: https://github.com/apache/arrow/pull/36034
(no comment)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] domoritz commented on pull request #36034: GH-36033: [JS] Remove BigInt compat
Posted by "domoritz (via GitHub)" <gi...@apache.org>.
domoritz commented on PR #36034:
URL: https://github.com/apache/arrow/pull/36034#issuecomment-1603379496
Seems like mostly old safari. I am good with dropping support going forward. People can use older versions of Arrow if they need the compat.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] conbench-apache-arrow[bot] commented on pull request #36034: GH-36033: [JS] Remove BigInt compat
Posted by "conbench-apache-arrow[bot] (via GitHub)" <gi...@apache.org>.
conbench-apache-arrow[bot] commented on PR #36034:
URL: https://github.com/apache/arrow/pull/36034#issuecomment-1613228121
Conbench analyzed the 6 benchmark runs on commit `7bc7ccaf`.
There were 3 benchmark results indicating a performance regression:
- Commit Run on `ursa-thinkcentre-m75q` at [2023-06-28 09:17:44Z](http://conbench.ursa.dev/compare/runs/56c06afdfa244f2991964c8586e3fe77...ae7d865cc21c4925a6d4f4773b317b23/)
- [params=<SMALL_VECTOR(int)>, source=cpp-micro, suite=arrow-small-vector-benchmark](http://conbench.ursa.dev/compare/benchmarks/0649bd3619ac784b8000d47ad0cf48d1...0649bfaf75cf7cd6800002b5c04726d6)
- [source=cpp-micro, suite=arrow-builder-benchmark](http://conbench.ursa.dev/compare/benchmarks/0649bd3a0e9b70d68000672e9346f2bd...0649bfb419537e7f8000ef6a5be909bf)
- and 1 more (see the report linked below)
The [full Conbench report](https://github.com/apache/arrow/runs/14657889201) has more details.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] trxcllnt commented on pull request #36034: GH-36033: [JS] Remove BigInt compat
Posted by "trxcllnt (via GitHub)" <gi...@apache.org>.
trxcllnt commented on PR #36034:
URL: https://github.com/apache/arrow/pull/36034#issuecomment-1603021568
I support this assuming we're fine with the matrix here: https://caniuse.com/?search=bigint64array
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] domoritz merged pull request #36034: GH-36033: [JS] Remove BigInt compat
Posted by "domoritz (via GitHub)" <gi...@apache.org>.
domoritz merged PR #36034:
URL: https://github.com/apache/arrow/pull/36034
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org