[ https://issues.apache.org/jira/browse/DRILL-3017?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Venki Korukanti updated DRILL-3017: ----------------------------------- Attachment: DRILL-3017-1.patch Given that a fragment can exit at any point in execution, it is better to safeguard against nulls in RecordReader.cleanups(). > NPE when cleaning up some RecordReader implementations > ------------------------------------------------------ > > Key: DRILL-3017 > URL: https://issues.apache.org/jira/browse/DRILL-3017 > Project: Apache Drill > Issue Type: Bug > Components: Execution - Relational Operators > Affects Versions: 0.9.0 > Reporter: Venki Korukanti > Assignee: Venki Korukanti > Attachments: DRILL-3017-1.patch > > > Run the following unittest: > {code} > @Test > public void testParquetReaderCleanupNPE() throws Exception { > test("SELECT * FROM cp.`parquet2/decimal28_38.parquet`"); > } > {code} > Following is the output: > {code} > Exception (no rows returned): > org.apache.drill.common.exceptions.UserRemoteException: SYSTEM ERROR: > Fragment 0:0 > [Error Id: 49db6650-8f62-4c5c-b9dc-3f5d6a4413a0 on localhost:31010]. > Returned in 407ms. > org.apache.drill.common.exceptions.UserRemoteException: SYSTEM ERROR: > Fragment 0:0 > {code} > Ideally in this case we should get the following query error: > {code} > Exception (no rows returned): > org.apache.drill.common.exceptions.UserRemoteException: UNSUPPORTED_OPERATION > ERROR: Decimal data type is disabled. > As of this release decimal data type is a beta level feature and should not > be used in production > Use option 'planner.enable_decimal_data_type' to enable decimal data type > Fragment 0:0 > [Error Id: d91a70ac-93c9-4be4-a542-4f3c7615b677 on localhost:31010]. > Returned in 392ms. > {code} -- This message was sent by Atlassian JIRA (v6.3.4#6332)