MINOR: Add explicit exception when no more buffer can be read when loading buffers

[viirya]

What's Changed

When VectorLoader tries to load buffers (i.e., loadBuffers), it has detect on the error case that some buffers are not consumed. But another error that the number of buffers is less than expected is not handled for now. Once it is happened, users will get java.util.NoSuchElementException which is not easy to understand.

This patch adds an explicit exception for such case.

See more discussion at #648.

[viirya]

This request has been automatically failed because it uses a deprecated version of `actions/cache: 6849a6489940f00c2f30c0fb92c6274307ccb58a`. Please update your workflow to use v3/v4 of actions/cache to avoid interruptions. 

Seems CI has some issues? I may not be able to handle it for now. Leave it to others or I can take a look tomorrow. Thanks.

[jbonofre]

FYI, I'm working on a PR about GitHub Action. It will be available soon and this PR can be rebase after the merge of my PR.

[viirya]

FYI, I'm working on a PR about GitHub Action. It will be available soon and this PR can be rebase after the merge of my PR.

Thank you @jbonofre

[jbonofre]

See #650

[viirya]

Rebased.

[viirya]

Thanks @lidavidm @jbonofre

One question. How do you merge PRs in this repo? Just use merge button or there is a script for it?

[jbonofre]

@viirya we are using "merge button", but only committers/PMC members can do that.

[viirya]

Thank you @jbonofre. I merged this.