[GitHub] [drill] paul-rogers commented on issue #1872: DRILL-7402: Suppress batch dumps for expected failures in tests

2019-10-13 Thread GitBox
paul-rogers commented on issue #1872: DRILL-7402: Suppress batch dumps for expected failures in tests URL: https://github.com/apache/drill/pull/1872#issuecomment-541466029 @arina-ielchiieva, this is a nice-to-have that makes running unit tests just a bit cleaner.

[GitHub] [drill] paul-rogers opened a new pull request #1872: DRILL-7402: Suppress batch dumps for expected failures in tests

2019-10-13 Thread GitBox
paul-rogers opened a new pull request #1872: DRILL-7402: Suppress batch dumps for expected failures in tests URL: https://github.com/apache/drill/pull/1872 Drill provides a way to dump the last few batches when an error occurs. However, in tests, we often deliberately cause something

[GitHub] [drill] paul-rogers commented on issue #1871: DRILL-7403: lidate batch checks, vector integretity in unit tests

2019-10-13 Thread GitBox
paul-rogers commented on issue #1871: DRILL-7403: lidate batch checks, vector integretity in unit tests URL: https://github.com/apache/drill/pull/1871#issuecomment-541451748 @arina-ielchiieva, please review. As we discussed, the idea here is to add the "vector checker", but with it

[GitHub] [drill] paul-rogers opened a new pull request #1871: DRILL-7403: lidate batch checks, vector integretity in unit tests

2019-10-13 Thread GitBox
paul-rogers opened a new pull request #1871: DRILL-7403: lidate batch checks, vector integretity in unit tests URL: https://github.com/apache/drill/pull/1871 Enhances the existing record batch checks to check all the various batch record counts, and to more fully validate all vector

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334292330 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334290031 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelFormatPlugin.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334291873 ## File path: protocol/src/main/java/org/apache/drill/exec/proto/UserBitShared.java ## @@ -980,6 +988,7

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334289892 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelFormatPlugin.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334291771 ## File path: contrib/format-excel/src/test/java/org/apache/drill/exec/store/excel/TestExcelFormat.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334289652 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelFormatConfig.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334292217 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334292681 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334289768 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelFormatConfig.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334292050 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334289963 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelFormatPlugin.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334291928 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334289832 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelFormatConfig.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334292021 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334292293 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334292168 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334289739 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelFormatConfig.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334291989 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334291952 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334292313 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334289619 ## File path: contrib/format-excel/pom.xml ## @@ -0,0 +1,112 @@ + + +http://maven.apache.org/POM/4.0.0;

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334292642 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334289669 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelFormatConfig.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334289680 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelFormatConfig.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334292088 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1749: DRILL-7177: Format Plugin for Excel Files URL: https://github.com/apache/drill/pull/1749#discussion_r334292700 ## File path: contrib/format-excel/src/main/java/org/apache/drill/exec/store/excel/ExcelBatchReader.java

[jira] [Created] (DRILL-7403) Validate batch checks, vector integretity, in unit tests

2019-10-13 Thread Paul Rogers (Jira)
Paul Rogers created DRILL-7403: -- Summary: Validate batch checks, vector integretity, in unit tests Key: DRILL-7403 URL: https://issues.apache.org/jira/browse/DRILL-7403 Project: Apache Drill

[jira] [Created] (DRILL-7402) Suppress batch dumps for expected failures in tests

2019-10-13 Thread Paul Rogers (Jira)
Paul Rogers created DRILL-7402: -- Summary: Suppress batch dumps for expected failures in tests Key: DRILL-7402 URL: https://issues.apache.org/jira/browse/DRILL-7402 Project: Apache Drill Issue

[GitHub] [drill] paul-rogers commented on issue #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-10-13 Thread GitBox
paul-rogers commented on issue #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#issuecomment-541442192 On, read, we can access the `DICT` as a dictionary and as an array. This means we need to be a bit clever about designing

[GitHub] [drill] paul-rogers commented on issue #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-10-13 Thread GitBox
paul-rogers commented on issue #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#issuecomment-541440586 Let's continue to assume that `DICT` is, essentially, `DICT` and that we can think of the `DICT`, when writing, as a pair of

[GitHub] [drill] paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r334287471 ## File path: exec/vector/src/main/java/org/apache/drill/exec/record/metadata/DictBuilder.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r334287531 ## File path: exec/vector/src/main/java/org/apache/drill/exec/record/metadata/DictBuilder.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r334287912 ## File path: exec/vector/src/main/java/org/apache/drill/exec/record/metadata/DictBuilder.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r334287737 ## File path: exec/vector/src/main/java/org/apache/drill/exec/record/metadata/DictBuilder.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r334287799 ## File path: exec/vector/src/main/java/org/apache/drill/exec/record/metadata/DictBuilder.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r334287620 ## File path: exec/vector/src/main/java/org/apache/drill/exec/record/metadata/DictBuilder.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r334287167 ## File path: exec/vector/src/main/java/org/apache/drill/exec/record/metadata/DictBuilder.java

[GitHub] [drill] paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r334287580 ## File path: exec/vector/src/main/java/org/apache/drill/exec/record/metadata/DictBuilder.java

[GitHub] [drill] paul-rogers commented on issue #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-10-13 Thread GitBox
paul-rogers commented on issue #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#issuecomment-541439848 Going to review this in multiple stages. First, let's pin down the desired semantics of the DICT type. See code comments.

[GitHub] [drill] paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework

2019-10-13 Thread GitBox
paul-rogers commented on a change in pull request #1870: DRILL-7359: Add support for DICT type in RowSet Framework URL: https://github.com/apache/drill/pull/1870#discussion_r334287763 ## File path: exec/vector/src/main/java/org/apache/drill/exec/record/metadata/DictBuilder.java

[GitHub] [drill] paul-rogers commented on issue #1868: DRILL-7377: Nested schemas for dynamic EVF columns

2019-10-13 Thread GitBox
paul-rogers commented on issue #1868: DRILL-7377: Nested schemas for dynamic EVF columns URL: https://github.com/apache/drill/pull/1868#issuecomment-541437430 @arina-ielchiieva, commits squashed This is an automated message