Extended Vector Framework ready for use

Hi All, A previous note explained how Drill has added the "Extended Vector Framework" (also called the "Row Set Framework") to improve the user's experience with Drill. On of Drill's key contributions is "schema-on-read": Drill can make sense of many kinds of data files without the hassle of

[GitHub] [drill] paul-rogers commented on issue #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

paul-rogers commented on issue #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#issuecomment-501956556 Added the ability to specify the regex (and column schema) in the provided schema. Defined a table property for the regex.

[GitHub] [drill] paul-rogers commented on issue #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

paul-rogers commented on issue #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#issuecomment-501956415 This PR is now failing due to the Protobuf errors. Thanks @vvysotskyi for fixing them. I'll rebase onto that fix once it is

[GitHub] [drill] paul-rogers commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

paul-rogers commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293633490 ## File path: exec/java-exec/src/test/java/org/apache/drill/exec/store/log/TestLogReader.java

[GitHub] [drill] paul-rogers commented on issue #1806: DRILL-7292: Remove V1 and V2 text readers

paul-rogers commented on issue #1806: DRILL-7292: Remove V1 and V2 text readers URL: https://github.com/apache/drill/pull/1806#issuecomment-501926628 @arina-ielchiieva, thank you for the review. Please check if the revised option messages are clear. The test failures may be due to

[GitHub] [drill] arina-ielchiieva commented on issue #1806: DRILL-7292: Remove V1 and V2 text readers

arina-ielchiieva commented on issue #1806: DRILL-7292: Remove V1 and V2 text readers URL: https://github.com/apache/drill/pull/1806#issuecomment-501769723 Issue with protobufs is a bug, will be fixed in https://issues.apache.org/jira/browse/DRILL-7294.

[GitHub] [drill] arina-ielchiieva commented on issue #1808: DRILL-7294: Prevent generating java beans using protostuff to avoid overriding classes with the same simple name declared as nested in the p

arina-ielchiieva commented on issue #1808: DRILL-7294: Prevent generating java beans using protostuff to avoid overriding classes with the same simple name declared as nested in the proto files URL: https://github.com/apache/drill/pull/1808#issuecomment-501764354 +1, thanks for fixing

[GitHub] [drill] vvysotskyi opened a new pull request #1808: DRILL-7294: Prevent generating java beans using protostuff to avoid overriding classes with the same simple name declared as nested in the

vvysotskyi opened a new pull request #1808: DRILL-7294: Prevent generating java beans using protostuff to avoid overriding classes with the same simple name declared as nested in the proto files URL: https://github.com/apache/drill/pull/1808 This PR contains two commits: - the first

[jira] [Created] (DRILL-7294) Prevent generating java beans using protostuff to avoid overriding classes with the same simple name declared as nested in the proto files

Volodymyr Vysotskyi created DRILL-7294: -- Summary: Prevent generating java beans using protostuff to avoid overriding classes with the same simple name declared as nested in the proto files Key: DRILL-7294

[GitHub] [drill] arina-ielchiieva commented on issue #1806: DRILL-7292: Remove V1 and V2 text readers

arina-ielchiieva commented on issue #1806: DRILL-7292: Remove V1 and V2 text readers URL: https://github.com/apache/drill/pull/1806#issuecomment-501675713 Ran the unit tests, Functional and Advanced, there are four expected failures on Functional tests (asked @agozhiy to take a look).

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293324471 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293323024 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/store/log/LogBatchReader.java

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293325049 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293326946 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/store/log/LogBatchReader.java

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293327844 ## File path: exec/java-exec/src/test/java/org/apache/drill/exec/store/log/TestLogReader.java

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293324364 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293326614 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/store/log/LogBatchReader.java

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293323517 ## File path:

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293327227 ## File path:

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r29332 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/store/log/LogBatchReader.java

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293323694 ## File path:

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293323284 ## File path:

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293326805 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/store/log/LogBatchReader.java

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293327274 ## File path:

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF

arina-ielchiieva commented on a change in pull request #1807: DRILL-7293: Convert the regex ("log") plugin to use EVF URL: https://github.com/apache/drill/pull/1807#discussion_r293324511 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1806: DRILL-7292: Remove V1 and V2 text readers

arina-ielchiieva commented on a change in pull request #1806: DRILL-7292: Remove V1 and V2 text readers URL: https://github.com/apache/drill/pull/1806#discussion_r293320197 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java ## @@ -713,11

[GitHub] [drill] arina-ielchiieva commented on a change in pull request #1806: DRILL-7292: Remove V1 and V2 text readers

arina-ielchiieva commented on a change in pull request #1806: DRILL-7292: Remove V1 and V2 text readers URL: https://github.com/apache/drill/pull/1806#discussion_r293320242 ## File path: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java ## @@ -713,11

[GitHub] [drill] paul-rogers commented on issue #1806: DRILL-7292: Remove V1 and V2 text readers

paul-rogers commented on issue #1806: DRILL-7292: Remove V1 and V2 text readers URL: https://github.com/apache/drill/pull/1806#issuecomment-501565970 Travis build failed due to protobuf changes. However, this PR changed no protobufs: neither a .proto file or the generated .java files.