Github user dprofeta commented on the issue:
https://github.com/apache/drill/pull/949
Indeed, in the plan, there is an added information that is the number of
rowgroup scanned. I think the check of the plan should be fixed by checking
separately numFiles and usedMetadataFile without e
Github user cchang738 commented on the issue:
https://github.com/apache/drill/pull/949
There is a plan verification failure due to plan change. The plan baseline
needs to be changed after this PR is merged.
Plan Verification Failures:
/root/drillAutomation/mapr/framework/r
Github user priteshm commented on the issue:
https://github.com/apache/drill/pull/949
@paul-rogers, @kkhatua can you provide some more information on the test
case that failed? Hopefully, @dprofeta can replicate it in his environment.
---
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/949
This change causes one of our functional tests to fail. We will have to
track down the issue and either update the test, or post the problem here.
---
Github user dprofeta commented on the issue:
https://github.com/apache/drill/pull/949
@paul-rogers done
---
Github user paul-rogers commented on the issue:
https://github.com/apache/drill/pull/949
Please resolve commits and rebase onto master.
---
Github user dprofeta commented on the issue:
https://github.com/apache/drill/pull/949
I have updated the patch with a unit test and fixed an issue when
everything is filtered.
---
Github user dprofeta commented on the issue:
https://github.com/apache/drill/pull/949
I will add a unit test to test the number of rowgroups that are scanned by
the groupscan to see if the filter is well able to prune rowgroup.
---
Github user dprofeta commented on the issue:
https://github.com/apache/drill/pull/949
@parthchandra Can you please review?
---
