You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@rya.apache.org by ejwhite922 <gi...@git.apache.org> on 2017/05/31 17:38:00 UTC
[GitHub] incubator-rya pull request #163: RYA-285 Fixed integration tests that use Mi...
GitHub user ejwhite922 opened a pull request:
https://github.com/apache/incubator-rya/pull/163
RYA-285 Fixed integration tests that use MiniAccumuloClusterInstance on Windows
## Description
Copied hadoop files to temp dir so that MiniAccumuloClusterInstance could run on Windows.
### Tests
Unit Tests
### Links
[Jira](https://issues.apache.org/jira/browse/RYA-285)
### Checklist
- [ ] Code Review
- [ ] Squash Commits
#### People To Review
@isper3at
@DLotts
@amihalik
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ejwhite922/incubator-rya RYA-285_MiniAccumuloClusterInstanceWindows
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-rya/pull/163.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 #163
----
commit 4fe9f44e7ab3ab608ab6c8591cf867c5fb52f4da
Author: eric.white <er...@parsons.com>
Date: 2017-05-31T17:35:02Z
RYA-285 Fixed integration tests that use MiniAccumuloClusterInstance on Windows
----
---
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-rya issue #163: RYA-285 Fixed integration tests that use MiniAccum...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit commented on the issue:
https://github.com/apache/incubator-rya/pull/163
Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/224/<h2>Build result: ABORTED</span></h2>[...truncated 6.86 MB...][INFO] Exclude: **/resources/META-INF/services/**[INFO] 4 resources included (use -debug for more details)[INFO] Rat check: Summary of files. Unapproved: 0 unknown: 0 generated: 0 approved: 4 licence.[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: java.util.concurrent.ExecutionException: Invalid object ID 12 iota=54[INFO] [INFO] --- maven-enforcer-plugin:1.3.1:enforce (enforce-mvn) @ rya.pcj.fluo.demo ---[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: java.util.concurrent.ExecutionException: Invalid object ID 12 iota=54[INFO] [INFO] --- maven-remote-resources-plugin:1.5:process (default) @ rya.pcj.fluo.demo ---[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: java.util.concurrent.ExecutionException: Invalid object ID 12 iota=54[INFO] [INFO] --- maven-resource
s-plugin:2.7:resources (default-resources) @ rya.pcj.fluo.demo ---[INFO] Using 'UTF-8' encoding to copy filtered resources.[INFO] skip non existing resourceDirectory /home/jenkins/jenkins-slave/workspace/incubator-rya-master-with-optionals-pull-requests/extras/rya.pcj.fluo/pcj.fluo.demo/src/main/resources[INFO] Copying 3 resources[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: java.util.concurrent.ExecutionException: Invalid object ID 12 iota=54[INFO] [INFO] --- maven-compiler-plugin:3.2:compile (default-compile) @ rya.pcj.fluo.demo ---[INFO] Changes detected - recompiling the module![INFO] Compiling 3 source files to /home/jenkins/jenkins-slave/workspace/incubator-rya-master-with-optionals-pull-requests/extras/rya.pcj.fluo/pcj.fluo.demo/target/classesBuild was abortedchannel stoppedSetting status of 4fe9f44e7ab3ab608ab6c8591cf867c5fb52f4da to FAILURE with url https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/224/ and message:
'Build finished. 'Using context: Jenkins: clean package -Pgeoindexing
---
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-rya pull request #163: RYA-285 Fixed integration tests that use Mi...
Posted by ejwhite922 <gi...@git.apache.org>.
Github user ejwhite922 closed the pull request at:
https://github.com/apache/incubator-rya/pull/163
---
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-rya pull request #163: RYA-285 Fixed integration tests that use Mi...
Posted by kchilton2 <gi...@git.apache.org>.
Github user kchilton2 commented on a diff in the pull request:
https://github.com/apache/incubator-rya/pull/163#discussion_r119745206
--- Diff: dao/accumulo.rya/src/test/java/org/apache/rya/accumulo/MiniAccumuloClusterInstance.java ---
@@ -76,6 +80,34 @@ public void stopMiniAccumulo() throws IOException, InterruptedException {
}
/**
+ * Copies the HADOOP_HOME bin directory to the {@link MiniAccumuloCluster} temp directory.
+ * {@link MiniAccumuloCluster} expects to find bin/winutils.exe in the MAC temp
+ * directory instead of HADOOP_HOME for some reason.
+ * @throws IOException
+ */
+ private static void copyHadoopHomeToTemp() throws IOException {
--- End diff --
So, this isn't the only class that spins up a Mini Accumulo. I think all of the Fluo tests are using a recipe that Fluo puts out, and that also uses Mini Accumulo. Do all of those tests pass on Windows also?
---
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-rya issue #163: RYA-285 Fixed integration tests that use MiniAccum...
Posted by jwonders <gi...@git.apache.org>.
Github user jwonders commented on the issue:
https://github.com/apache/incubator-rya/pull/163
I've been doing my development on Windows and can run the integration tests without code changes. If the `$HADOOP_HOME/bin` directory is on the windows `PATH` the MAC should be able to start. I still see some warnings, but they are not fatal issues.
---
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.
---