AlenkaF commented on code in PR #51404:
URL: https://github.com/apache/arrow/pull/51404#discussion_r4142778478
##########
python/pyarrow/tests/test_pandas.py:
##########
@@ -99,8 +99,8 @@ def _alltypes_example(size=100):
def _check_pandas_roundtrip(df, expected=None, use_threads=False,
expected_schema=None,
- check_dtype=True, schema=None,
- preserve_index=False,
+ check_dtype=True, check_freq=False,
Review Comment:
I was not sure this needs to be addressed, but you asking means it should be
;)
My thought process was: `check_freq` will be tested by default and as we do
not keep freq information from pandas (AFAIU) we might default to keeping the
current behavior which is not checking the frequency in any of our roundtrips.
Have no problem to change this as suggested though.
--
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]
