You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by dr-wolf <gi...@git.apache.org> on 2016/02/04 14:01:50 UTC

[GitHub] drill pull request: Storage jdbc oracle test

GitHub user dr-wolf opened a pull request:

    https://github.com/apache/drill/pull/357

    Storage jdbc oracle test

    Created test class for oracle db and oracle sample data set. Updated dependencies in storage-jdbc/pom.xml, make sure that ojdbc7.jar is installed in maven.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/dr-wolf/drill storage-jdbc-oracle-test

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/drill/pull/357.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 #357
    
----
commit 8fbdb9a03db5a66bcf5c0c287e9d0bb23d421f53
Author: dr-wolf <ta...@dremio.com>
Date:   2016-02-04T12:48:43Z

    Oracle JDBC test suit

commit 0fb251b3a02b09c44f4d6d717d9ef06abc0e263f
Author: dr-wolf <ta...@dremio.com>
Date:   2016-02-04T12:55:28Z

    Drill func pushdown test

commit d02bb019bd98c61efe4d06ce31b3db30eefa5cb6
Author: dr-wolf <ta...@dremio.com>
Date:   2016-02-04T13:00:52Z

    Driver config updated

----


---
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] drill pull request #357: Storage jdbc oracle test

Posted by dr-wolf <gi...@git.apache.org>.
Github user dr-wolf closed the pull request at:

    https://github.com/apache/drill/pull/357


---

[GitHub] drill pull request: Storage jdbc oracle test

Posted by jacques-n <gi...@git.apache.org>.
Github user jacques-n commented on the pull request:

    https://github.com/apache/drill/pull/357#issuecomment-179896052
  
    Since Oracle must be running and it isn't an Apache licensed file, we need to make this test and dependency only run with a special profile.


---
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] drill pull request: Storage jdbc oracle test

Posted by dr-wolf <gi...@git.apache.org>.
Github user dr-wolf commented on the pull request:

    https://github.com/apache/drill/pull/357#issuecomment-180362125
  
    I have excluded oracle tests and dependencies from default profile and created oracle profile, but I still have some problems with running verify with this profile and I'm working on it.


---
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.
---