AlenkaF commented on PR #34730: URL: https://github.com/apache/arrow/pull/34730#issuecomment-1503088155
Thank you for the contribution @mikelui! The Python part of the code looks good to me. There are two long lines to be corrected: ```python /arrow/python/pyarrow/tests/test_pandas.py:4610:89: E501 line too long (95 > 88 characters) /arrow/python/pyarrow/tests/test_pandas.py:4627:89: E501 line too long (92 > 88 characters) ``` As for the C++ part and the question if this is a good option to have or not, Joris will probably have a better comment and ideas. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
