jorisvandenbossche commented on code in PR #50769:
URL: https://github.com/apache/arrow/pull/50769#discussion_r3784767562


##########
python/pyarrow/tests/parquet/test_pandas.py:
##########
@@ -430,16 +429,14 @@ def 
test_backwards_compatible_index_multi_level_some_named(datadir):
 
 @pytest.mark.pandas
 def test_backwards_compatible_column_metadata_handling(datadir):

Review Comment:
   For a normal (non-multi) index, pandas already raised before in 
`assert_frame_equal` (and so there the referenced PR is not changing anything), 
but for a MultiIndex I indeed see a change in behaviour, and ideally that 
should go through a deprecation warning as well.
   
   But indeed, for here, passing `check_freq=False` should resolve it. Or 
alternatively add a `._with_freq(None)` to the chain when creating the `dates` 
variable.



-- 
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]

Reply via email to