-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
ARROW-16431: [C++][Python] Improve AppendRowGroups error when schemas differ #14029
ARROW-16431: [C++][Python] Improve AppendRowGroups error when schemas differ #14029
Conversation
Can you rebase once more to see if that makes the failures go away? |
- Refactor diff output to use ostream* - Update diff message - Fix import ordering
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.
+1. Thanks for this @milesgranger !
Benchmark runs are scheduled for baseline = 35bfeb4 and contender = 6cccec5. 6cccec5 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
['Python', 'R'] benchmarks have high level of regressions. |
… differ (apache#14029) Fix [ARROW-16431](https://issues.apache.org/jira/browse/ARROW-16431) Feel free to opine on specific error messages or the implementation as a whole. 👌 Examples ```python # meta1 and meta2 differ in column types meta1.append_row_groups(meta2) *** RuntimeError: AppendRowGroups requires equal schemas. The two columns with index 0 differ. column descriptor = { name: col1, path: col1, physical_type: INT64, converted_type: NONE, logical_type: None, max_definition_level: 1, max_repetition_level: 0, } column descriptor = { name: col2, path: col2, physical_type: INT64, converted_type: NONE, logical_type: None, max_definition_level: 1, max_repetition_level: 0, } # meta1 and meta2 differ in number of columns meta1.append_row_groups(meta2) *** RuntimeError: This schema has 2 columns, other has 1 ``` Authored-by: Miles Granger <miles59923@gmail.com> Signed-off-by: Antoine Pitrou <antoine@python.org>
Fix ARROW-16431
Feel free to opine on specific error messages or the implementation as a whole. 👌
Examples