-
Notifications
You must be signed in to change notification settings - Fork 3.9k
GH-48448: [Python] Implement Alphanumeric and Surrogate text in the random schema generator #48449
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
base: main
Are you sure you want to change the base?
Conversation
| max_codepoint=0x7E | ||
| ) | ||
| ) | ||
|
|
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.
|
|
0f436c0 to
de8e939
Compare
|
@github-actions crossbow submit -g python |
AlenkaF
left a comment
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.
|
(just for a bit of context, I am fixing hypothesis at #48460 as it's broken, and @raulcd guided me to work on this at #48443 (review) 👍) |
|
Revision: de8e939 Submitted crossbow builds: ursacomputing/crossbow @ actions-15cc39ef2c |
|
Oh yes! Forgot about that, thanks for the info 👍 |
de8e939 to
1a13b48
Compare
|
@github-actions crossbow submit -g python |
|
Revision: 1a13b48 Submitted crossbow builds: ursacomputing/crossbow @ actions-12f77fab36 |
1a13b48 to
1e3016a
Compare
|
Rebased to retrigger the tests. #48449 (comment) is still valid. test-conda-python-3.11-hypothesis passed |
|
Seems like: Failure at MacOS is globally happening. I retriggered but still the issue persists. Let me leave it as is for now - it won't be related to my change in any event. |
|
Yes, macOS failures are not related. Thanks for all the work @HyukjinKwon! |
Rationale for this change
To test various cases of field names:
arrow/python/pyarrow/tests/strategies.py
Line 49 in 6456944
It was introduced from 9da4584
What changes are included in this PR?
This PR implements Alphanumeric and Surrogate text in the random schema generator
Are these changes tested?
Yes I tested them via:
Are there any user-facing changes?
No, test-only.