[BUG] is_datetime_or_timedelta removed in Pandas 2.1.0

[dtaivpp]

What is the bug?
We import is_datetime_or_timedelta in field_mappings.py but that has been removed from Pandas version 2.1.0.

How can one reproduce the bug?

1. pip install pandas==2.1.0
2. from opensearch_py_ml.ml_commons import MLCommonClient
3. run the file

What is the expected behavior?
Should not give import error.

What is your host/environment?
N/A

Do you have any screenshots?
If applicable, add screenshots to help explain your problem.

Do you have any additional context?
Add any other context about the problem.

[bsowell]

I was bit by this one recently. @dtaivpp @dhrubo-os I'm not familiar with the CCI label -- is this one somebody is working on, or would you take a PR?

[dhrubo-os]

I was bit by this one recently. @dtaivpp @dhrubo-os I'm not familiar with the CCI label -- is this one somebody is working on, or would you take a PR?

Currently nobody is working on this issue. I marked this as CCI so that our next college contributors can work on this issue. PRs are welcome.

[bsowell]

Got it. Thanks! Let me look into it and see if I can contribute something.

[rawwar]

I see that a quick fix for this is to limit the version of pandas to < 2.0 .

[dhrubo-os]

I see that a quick fix for this is to limit the version of pandas to < 2.0 .

Yeah, we can just do: pandas>=1.5.2,<2.0 for a quick fix.

@bsowell are you working on this issue? Just checking.

[bsowell]

I'm sorry. I had trouble getting tests to run locally and then got distracted so I haven't done much. The implementation of is_datetime_or_timedelta_dtype is straightforward, so if there is value in upgrading pandas, it would be easy enough to just include the logic in field_mappings.py, but agree that the dependency change would be faster.

[Yerzhaisang]

I think we should close this issue. is_datetime_or_timedelta is not used anymore

[dhrubo-os]

Closing this issue in favor of #316