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

TAJO-1904: Extend GraphVisitor to accept user-defined context#797

Closed
jihoonson wants to merge 6 commits into
apache:masterfrom
jihoonson:TAJO-1904
Closed

TAJO-1904: Extend GraphVisitor to accept user-defined context#797
jihoonson wants to merge 6 commits into
apache:masterfrom
jihoonson:TAJO-1904

Conversation

@jihoonson
Copy link
Copy Markdown
Contributor

No description provided.

@hyunsik
Copy link
Copy Markdown
Member

hyunsik commented Oct 5, 2015

Could you rebase it?

@jihoonson
Copy link
Copy Markdown
Contributor Author

Rebased.

…into TAJO-1904

Conflicts:
	tajo-core/src/main/java/org/apache/tajo/engine/planner/global/rewriter/rules/BroadcastJoinRule.java
@hyunsik
Copy link
Copy Markdown
Member

hyunsik commented Nov 2, 2015

Hi @jihoonson,

+1
The patch looks good to me. This feature has been necessary. I manually verified 'mvn clean install'. All unit tests are passed.

@asfgit asfgit closed this in 3224add Nov 3, 2015
@jihoonson
Copy link
Copy Markdown
Contributor Author

@hyunsik, thanks for your review. I've committed.

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