Code clean for new datafusion-cli streaming printing logic

[zhuqi-lucas]

Is your feature request related to a problem or challenge?

This is a follow-up for:
#14877 (comment)

And we can do some code clean work including:

1. Add streaming state struct and clean up print batch logic
2. Remove older logics about max-rows
3. Add more docs and remove confusing docs
4. Etc

Describe the solution you'd like

No response

Describe alternatives you've considered

No response

Additional context

No response

[alamb]

I think this is a nice first isuse for someone who wants to make the code nicer and it is currently tested and could be handled without deep datafusion knowledge

It does require some non trivial rust / refactoring though so it probably isn't a great one for someone new to rust

[shruti2522]

Hi @zhuqi-lucas , I would like to work on this issue.

[shruti2522]

take

[zhuqi-lucas]

Sure @shruti2522 , thanks!

[alamb]

Thanks @shruti2522 -- can't wait to see what you come up with ❤

[THEAYUSHIMISHRA]

Hi @zhuqi-lucas , I would like to work on this issue

[alamb]

I think the current status is that the code for this feature was somewhat complicated and in fact caused a regression before the last release so we reverted it.

@zhuqi-lucas has made a new PR with some more tests, but the code is still somewhat complex and thus hard to review. Perhaps you can help review / improve that pR?

• feat: Fix multi-lines printing issue for datafusion-cli and add the streaming printing feature back #14954