Add row ID support to batch ORC reader

[elharo]

Description

Previously only done in selective reader

Motivation and Context

Fill in row IDs on more paths

Impact

none

Test Plan

CI

Contributor checklist

• Please make sure your submission complies with our development, formatting, commit message, and attribution guidelines.
• PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
• Documented new properties (with its default value), SQL syntax, functions, or other functionality.
• If release notes are required, they follow the release notes guidelines.
• Adequate tests were added if applicable.
• CI passed.

Release Notes

== NO RELEASE NOTE ==

[sdruzkin]

LGTM. Might want to add more tests.

[NikhilCollooru]

Can we add some unit tests ?

[NikhilCollooru]

There are 5 checkstyle errors. please check.

[INFO] There are 5 errors reported by Checkstyle 8.16 with src/checkstyle/presto-checks.xml ruleset.
Error:  src/main/java/com/facebook/presto/hive/HiveUtil.java:[216,127] (blocks) LeftCurly: '{' at column 127 should be on a new line.
Error:  src/main/java/com/facebook/presto/hive/orc/DwrfSelectivePageSourceFactory.java:[54,15] (imports) UnusedImports: Unused import - com.google.common.base.Preconditions.checkArgument.
Error:  src/test/java/com/facebook/presto/hive/TestHiveUtil.java:[90,60] (blocks) LeftCurly: '{' at column 60 should be on a new line.
Error:  src/test/java/com/facebook/presto/hive/TestHiveUtil.java:[97,60] (blocks) LeftCurly: '{' at column 60 should be on a new line.
Error:  src/test/java/com/facebook/presto/hive/TestHiveUtil.java:[104,58] (blocks) LeftCurly: '{' at column 58 should be on a new line.

[elharo]

PTAL. Adding unit tests invalidated existing approvals.

[NikhilCollooru]

The test added in this PR is failing . please check

Method TestHivePageSourceProvider.testCreatePageSource_withRowIDMissingPartitionComponent()[pri:0, instance:com.facebook.presto.hive.TestHivePageSourceProvider@c017175] should have thrown an exception of type class java.lang.IllegalArgumentException
	at org.testng.internal.invokers.ExpectedExceptionsHolder.noException(ExpectedExceptionsHolder.java:81)
	at org.testng.internal.invokers.TestInvoker.considerExceptions(TestInvoker.java:853)
	at org.testng.internal.invokers.TestInvoker.invokeMethod(TestInvoker.java:714)
	at org.testng.internal.invokers.TestInvoker.invokeTestMethod(TestInvoker.java:220)
	at org.testng.internal.invokers.MethodRunner.runInSequence(MethodRunner.java:50)
	at org.testng.internal.invokers.TestInvoker$MethodInvocationAgent.invoke(TestInvoker.java:945)
	at org.testng.internal.invokers.TestInvoker.invokeTestMethods(TestInvoker.java:193)
	at org.testng.internal.invokers.TestMethodWorker.invokeTestMethods(TestMethodWorker.java:146)
	at org.testng.internal.invokers.TestMethodWorker.run(TestMethodWorker.java:128)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:750)

[INFO] 
[INFO] Results:
[INFO] 
Error:  Failures: 
Error:    TestHivePageSourceProvider.testCreatePageSource_withRowIDMissingPartitionComponent » Test 
Method TestHivePageSourceProvider.testCreatePageSource_withRowIDMissingPartitionComponent()[pri:0, instance:com.facebook.presto.hive.TestHivePageSourceProvider@c017175] should have thrown an exception of type class java.lang.IllegalArgumentException
[INFO] 
Error:  Tests run: 2976, Failures: 1, Errors: 0, Skipped: 90

[elharo]

Fixed. Unfortunately that invalidates existing approvals. :-(

[elharo]

Remaining failure looks flaky, rerunning