Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

aot the cascalog.logic.def #248

Closed
wants to merge 1 commit into from

Conversation

methylene
Copy link

Fix the issue 245 "ClassNotFoundException: cascalog.logic.def.ParallelAggregator" by aot-compiling the cascalog.logic.def namespace.

@sritchie
Copy link
Collaborator

Hmm, adding AOT into a library is bad form. Under what circumstances is this bug showing up?—
Sent from Mailbox

On Sun, Apr 27, 2014 at 2:13 AM, methylene notifications@github.com
wrote:

Fix the issue 245 "ClassNotFoundException: cascalog.logic.def.ParallelAggregator" by aot-compiling the cascalog.logic.def namespace.
You can merge this Pull Request by running:
git pull https://github.com/methylene/cascalog issue-245-aot
Or you can view, comment on it, or merge it online at:
#248
-- Commit Summary --

@sritchie
Copy link
Collaborator

Option you need to require a namespace before importing the java class some record is meant to generate. Try adding an explicit require to the project giving you the problem.—
Sent from Mailbox

On Sun, Apr 27, 2014 at 12:49 PM, Sam Ritchie sritchie09@gmail.com
wrote:

Hmm, adding AOT into a library is bad form. Under what circumstances is this bug showing up?—
Sent from Mailbox
On Sun, Apr 27, 2014 at 2:13 AM, methylene notifications@github.com
wrote:

Fix the issue 245 "ClassNotFoundException: cascalog.logic.def.ParallelAggregator" by aot-compiling the cascalog.logic.def namespace.
You can merge this Pull Request by running:
git pull https://github.com/methylene/cascalog issue-245-aot
Or you can view, comment on it, or merge it online at:
#248
-- Commit Summary --

@methylene
Copy link
Author

I tried to do it here, but it's still break: methylene/class-not-found@8689ae3

@methylene
Copy link
Author

Actually there seems to be a more general problem with leiningen, see methylene/class-not-found@902dfb3

@sritchie
Copy link
Collaborator

Okay, in that case, closing this up.

@sritchie sritchie closed this Apr 29, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants