[DRILL-5587] Validate Parquet blockSize and pageSize configured with SYSTEM/SESSION option

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.10.0
• Fix Version/s: 1.11.0
• Component/s: Storage - Parquet
• Labels: ready-to-commit
• Assignee: Padma Penumarthy
• Reporter: Padma Penumarthy
• Reviewer: Paul Rogers
• Created: 14/Jun/17 05:10
• Updated: 06/Jul/17 10:17
• Resolved: 06/Jul/17 10:17

Description

We can set Parquet blockSize, pageSize and dictionary pageSize to any value. It uses LongValidator which is not exactly validating the value. Since all these sizes are used as int in the code, even though user is able to set them to any value (could be greater than MAXINT and/or negative), parsing the value later in the code as int can throw an error. Instead, restrict the value that can be set to MAXINT.
There is a bug open for validating system/session options in general. https://issues.apache.org/jira/browse/DRILL-2478

Issue Links

• links to: GitHub Pull Request #852