Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

TAJO-1823: Can't start TajoMaster.#740

Closed
hyunsik wants to merge 1 commit into
apache:masterfrom
hyunsik:TAJO-1823
Closed

TAJO-1823: Can't start TajoMaster.#740
hyunsik wants to merge 1 commit into
apache:masterfrom
hyunsik:TAJO-1823

Conversation

@hyunsik
Copy link
Copy Markdown
Member

@hyunsik hyunsik commented Sep 7, 2015

No description provided.

@jinossy
Copy link
Copy Markdown
Member

jinossy commented Sep 7, 2015

Thanks for your quick fix but I can't compile on IDE
The problem looks like that incorrect generated source path and module order.

Could you create the package directory ?

tajo-sql-parser/src/main/antlr4/org.apache.tajo.parser.sql

Please change module order as follows :

...
<module>tajo-algebra</module>
<module>tajo-sql-parser</module>
...

@hyunsik
Copy link
Copy Markdown
Member Author

hyunsik commented Sep 7, 2015

Thank you for the comment. I'll check out them.

@hyunsik
Copy link
Copy Markdown
Member Author

hyunsik commented Sep 7, 2015

I fixed the wrong package name and rebased the patch.

@jinossy
Copy link
Copy Markdown
Member

jinossy commented Sep 7, 2015

+1

@asfgit asfgit closed this in b1d08f8 Sep 7, 2015
asfgit pushed a commit that referenced this pull request Sep 7, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants