You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by vkorukanti <gi...@git.apache.org> on 2014/04/16 19:37:20 UTC
[GitHub] incubator-drill pull request: DRILL-533: Queries fail with NPE if ...
GitHub user vkorukanti opened a pull request:
https://github.com/apache/incubator-drill/pull/51
DRILL-533: Queries fail with NPE if dfs/cp schema has no default workspa...
...ce but has non-default workspace
We currently add the "default" workspace only if there are no workspaces defined, but we always choose the workspace with "default" name as the default workspace. As in the repro case there is no default workspace, we end with a null.
Fix: Add default workspace if there is no "default" given in storage-engines.json
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vkorukanti/incubator-drill DRILL-533
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-drill/pull/51.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #51
----
commit 9f6ec7783d6b97d65561c23914d06d0df7b6b33d
Author: vkorukanti <ve...@gmail.com>
Date: 2014-04-16T17:32:24Z
DRILL-533: Queries fail with NPE if dfs/cp schema has no default workspace but has non-default workspace
We currently add the "default" workspace only if there are no workspaces defined, but we always choose the workspace with "default" name as the default workspace. As in the repro case there is no default workspace, we end with a null.
Fix: Add default workspace if there is no "default" given in storage-engines.json
----
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-drill pull request: DRILL-533: Queries fail with NPE if ...
Posted by vkorukanti <gi...@git.apache.org>.
Github user vkorukanti closed the pull request at:
https://github.com/apache/incubator-drill/pull/51
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-drill pull request: DRILL-533: Queries fail with NPE if ...
Posted by vkorukanti <gi...@git.apache.org>.
Github user vkorukanti commented on the pull request:
https://github.com/apache/incubator-drill/pull/51#issuecomment-41185880
rebased on latest master and fixed an issue in earlier fix.
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-drill pull request: DRILL-533: Queries fail with NPE if ...
Posted by vkorukanti <gi...@git.apache.org>.
Github user vkorukanti commented on the pull request:
https://github.com/apache/incubator-drill/pull/51#issuecomment-41297171
merged as 7a4fee42406516e729b9bdae31945cec4858a332.
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---