You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by Fokko <gi...@git.apache.org> on 2016/12/27 15:10:26 UTC
[GitHub] flink pull request #3048: Clarified the import path of the Breeze DenseVecto...
GitHub user Fokko opened a pull request:
https://github.com/apache/flink/pull/3048
Clarified the import path of the Breeze DenseVector
Guys,
I'm working on an extension of the ml library on Flink, but I stumbled upon this. Since it is such a trivial fix, I didn't created a JIRA request. Keep up the good work!
Cheers,
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Fokko/flink fd-cleanup-package-structure
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/3048.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 #3048
----
commit 3fd38fe9785d607a05d045cd54a05af9ed48e350
Author: Fokko Driesprong <fo...@godatadriven.com>
Date: 2016-12-27T14:43:15Z
Replaced the full import path with the BreezeDenseVector itself to make it more readable
----
---
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] flink pull request #3048: Clarified the import path of the Breeze DenseVecto...
Posted by Fokko <gi...@git.apache.org>.
Github user Fokko closed the pull request at:
https://github.com/apache/flink/pull/3048
---
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] flink issue #3048: Clarified the import path of the Breeze DenseVector
Posted by Fokko <gi...@git.apache.org>.
Github user Fokko commented on the issue:
https://github.com/apache/flink/pull/3048
Locally the tests just pass. Looking at the error logs, it doesn't have to do with the changes in the PR, for example:
```
java.io.FileNotFoundException: build-target/lib/flink-dist-*.jar (No such file or directory)
at java.util.zip.ZipFile.open(Native Method)
at java.util.zip.ZipFile.<init>(ZipFile.java:220)
at java.util.zip.ZipFile.<init>(ZipFile.java:150)
at java.util.zip.ZipFile.<init>(ZipFile.java:121)
at sun.tools.jar.Main.list(Main.java:1060)
at sun.tools.jar.Main.run(Main.java:291)
at sun.tools.jar.Main.main(Main.java:1233)
find: `./flink-yarn-tests/target/flink-yarn-tests*': No such file or directory
```
---
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.
---