Return empty dataframe when reading an ORC file using empty columns option

[vuule]

Description

Changes are mostly equivalent to Parquet changes in #11018.

Store the columns option as optional:

• nullopt when columns are not passed by caller - read all columns.
• Empty vector when caller explicitly passes an empty list/vector - return empty dataframe.
• Vector of column names - read columns with given names.

Also includes a small cleanup of the code equivalent in the Parquet reader.

Fixes #11021

Checklist

• I am familiar with the Contributing Guidelines.
• New or existing tests cover these changes.
• The documentation is up to date with these changes.

[galipremsagar]

Overall changes LGTM, we will need the following workaround due to a bug in pandas.

[galipremsagar]

Good to go from cython/python side! Thanks @vuule !!

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (branch-22.10@e099e01). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             branch-22.10   #11446   +/-   ##
===============================================
  Coverage                ?   86.47%           
===============================================
  Files                   ?      144           
  Lines                   ?    22856           
  Branches                ?        0           
===============================================
  Hits                    ?    19765           
  Misses                  ?     3091           
  Partials                ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[mythrocks]

+1 for the C++ and Java/JNI code.

[ttnghia]

C++ approval.

[vuule]

@gpucibot merge