Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fix to_json for numbers larger than sys.maxsize #34473
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
fix to_json for numbers larger than sys.maxsize #34473
Changes from 15 commits
95c20db
6d2f8bd
4fc5b87
abfca37
7e63941
3353420
c9574b8
94c112f
76576b8
9f211a5
2b7a271
5e06109
755ef47
0e768f8
12d73b0
6c2aa9f
1a8051f
552194e
e2898ef
2943995
771ec5d
92bc6ef
8f3af8c
cdae92e
1009168
759ad8a
5e01ed0
8a08a38
0441fe7
4630c0d
2e06a8b
63056fc
6ec960e
c63a5c9
b2f8f46
fea9348
6516078
aa2dbca
7eaf42d
56d5bac
1cdb1ba
821d51f
1001ac1
b8f16b6
a6e83c7
ccc5b47
585b985
7586698
0e6768f
7c19bd2
9809d7c
2739f3d
77d69b7
f003d6b
ee505c9
cdc0870
0fba3d5
259018d
a856a41
1bbfdc2
665b146
3608297
4ab13d6
176f212
9b58758
44b79f1
9cbf596
7ee21eb
948170c
ff2e25e
ce37048
2db12c0
3e820ac
7afeadb
e4df0f8
7b041fe
21d9e98
6fd15c9
c7acef1
2d43001
6053227
a688468
9e1b95f
cc0dd6a
4e53974
5c96ae4
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you also add a full integration test in json/test_pandas.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should also parametrize this to exceed the minimum supported native size
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you do the parametrization of a large negative value here too?