[GitHub] drill pull request #584: DRILL-4884: Fix bug that drill sometimes produced I...

GitHub user zbdzzg opened a pull request: https://github.com/apache/drill/pull/584 DRILL-4884: Fix bug that drill sometimes produced IOB exception while querying data of 65536 limitation Drill produces IOB while using a non batched scanner and limiting SQL by 65536. SQL

[GitHub] drill issue #584: DRILL-4884: Fix bug that drill sometimes produced IOB exce...

Github user zbdzzg commented on the issue: https://github.com/apache/drill/pull/584 @dsbos --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] drill pull request #584: DRILL-4884: Fix bug that drill sometimes produced I...

Github user zbdzzg commented on a diff in the pull request: https://github.com/apache/drill/pull/584#discussion_r84835453 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/validate/IteratorValidatorBatchIterator.java --- @@ -301,7 +301,7 @@ public

[GitHub] drill pull request #663: DRILL-5054: Provide source files of jquery-ui and j...

Github user zbdzzg commented on a diff in the pull request: https://github.com/apache/drill/pull/663#discussion_r89448653 --- Diff: exec/java-exec/src/main/resources/rest/query/result.ftl --- @@ -11,14 +11,13 @@ <#include "*/generic.ftl"> &

[GitHub] drill pull request #663: DRILL-5054: Provide source files of jquery-ui and j...

Github user zbdzzg closed the pull request at: https://github.com/apache/drill/pull/663 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] drill pull request #668: DRILL-5068: Add a new system table for completed pr...

GitHub user zbdzzg opened a pull request: https://github.com/apache/drill/pull/668 DRILL-5068: Add a new system table for completed profiles Add table "sys.profiles" for completed queries. Following fields added: 1. queryID (String) 2. time (Timest

[GitHub] drill pull request #658: DRILL-5051: Fix incorrect result returned in nest q...

GitHub user zbdzzg opened a pull request: https://github.com/apache/drill/pull/658 DRILL-5051: Fix incorrect result returned in nest query with offset specified 1. Fix this problem. 2. Merge two limit methods, most of code is the same. You can merge this pull request

[GitHub] drill pull request #658: DRILL-5051: Fix incorrect result returned in nest q...

Github user zbdzzg closed the pull request at: https://github.com/apache/drill/pull/658 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] drill pull request #662: DRILL-5051: Fix incorrect result returned in nest q...

GitHub user zbdzzg opened a pull request: https://github.com/apache/drill/pull/662 DRILL-5051: Fix incorrect result returned in nest query with offset s… …pecified 1. Fix this problem. 2. Merge two limit methods, most of code is the same. You can merge this pull

[GitHub] drill pull request #663: DRILL-5054: Provide source files of jquery-ui and j...

GitHub user zbdzzg opened a pull request: https://github.com/apache/drill/pull/663 DRILL-5054: Provide source files of jquery-ui and jquery-dataTables locally Hi, Currently Drill uses CDN for serving source files of jquery-ui and jquery-dataTables. This is OK for most

[GitHub] drill issue #584: DRILL-4884: Fix bug that drill sometimes produced IOB exce...

Github user zbdzzg commented on the issue: https://github.com/apache/drill/pull/584 @jinfengni A test case has been added, See 2c67dd78e463514393fd33779b64fa88c7c22ebd --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] drill issue #662: DRILL-5051: Fix incorrect result returned in nest query wi...

Github user zbdzzg commented on the issue: https://github.com/apache/drill/pull/662 @sudheeshkatkam Could you please review this? Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] drill issue #662: DRILL-5051: Fix incorrect result returned in nest query wi...

Github user zbdzzg commented on the issue: https://github.com/apache/drill/pull/662 @sudheeshkatkam Thanks for your review, commit message has been changed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] drill pull request #662: DRILL-5051: Fix incorrect result returned in nest q...

Github user zbdzzg commented on a diff in the pull request: https://github.com/apache/drill/pull/662#discussion_r93163164 --- Diff: exec/java-exec/src/test/java/org/apache/drill/TestBugFixes.java --- @@ -221,4 +221,14 @@ public void testDRILL4884() throws Exception

[GitHub] drill pull request #668: DRILL-5068: Add a new system table for completed pr...

Github user zbdzzg closed the pull request at: https://github.com/apache/drill/pull/668 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] drill pull request #668: DRILL-5068: Add a new system table for completed pr...

GitHub user zbdzzg reopened a pull request: https://github.com/apache/drill/pull/668 DRILL-5068: Add a new system table for completed profiles Add table "sys.profiles" for completed queries. Following fields added: 1. queryID (String) 2. time (Timest