REFACTOR: remove redefinition of _row_lengths and _column_widths functions in PandasOnDaskDataframe.

[anmyachev]

Basic implementation should work for Dask engine. It already works for Ray engine.

Links to basic implementations:

modin/modin/core/dataframe/pandas/dataframe/dataframe.py

Line 105 in d590de0

def _row_lengths(self):

modin/modin/core/dataframe/pandas/dataframe/dataframe.py

Line 124 in d590de0

def _column_widths(self):

Links to Dask redefinitions:

modin/modin/core/execution/dask/implementations/pandas_on_dask/dataframe/dataframe.py

Line 47 in d590de0

def _row_lengths(self):

modin/modin/core/execution/dask/implementations/pandas_on_dask/dataframe/dataframe.py

Line 64 in d590de0

def _column_widths(self):

[vnlitvinov]

TBH, I would rather we add a specific Ray implementation that also gathers all objects widths in one go instead of doing a loop, this could potentially be a little bit faster.

[devin-petersohn]

It should be significantly faster @vnlitvinov.

[vnlitvinov]

I wonder if #4683 and #4718 actually address this. @anmyachev thoughts?

[anmyachev]

I wonder if #4683 and #4718 actually address this. @anmyachev thoughts?

Really! Now the default implementation is enough, which for both Dask and Ray gather all remote objects at once.