Third batch of updating sql tests to use assert_batches_eq

[matthewmturner]

Which issue does this PR close?

Continued work on #743 task 2.

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

[alamb]

Thank you @matthewmturner -- I think the reason this is failing is that the github executors have 2 cores, but your machine had 8, but the physical explain plans depend on the number of cores. I'll fix up the tests (as I have a vested interest in pushing this project along)

[matthewmturner]

@alamb noted, thanks!

it might be a silly question, but would you be able to provide color on why the physical explain plan depend on the number of cores?

[alamb]

it might be a silly question, but would you be able to provide color on why the physical explain plan depend on the number of cores?

It is not silly at all @matthewmturner -- the core reason is that the target concurrency defaults to the number of cores -- so when DataFusion goes through the exercise is splitting the input into several output datastreams (partitons), it tries to make just enough output streams to keep all the cores busy.