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

TAJO-1451: implement elasticsearch storage for tajo.#464

Closed
HowookJeong wants to merge 2 commits intoapache:masterfrom
HowookJeong:TAJO-1451
Closed

TAJO-1451: implement elasticsearch storage for tajo.#464
HowookJeong wants to merge 2 commits intoapache:masterfrom
HowookJeong:TAJO-1451

Conversation

@HowookJeong
Copy link

I'm sorry.
I retried my pull request and then please review this request.

@blrunner
Copy link
Contributor

Thanks for your contribution and I suggest you to consider following opinions.

First, I found that you had wrote your name and current date. But we don't write our name and some date at all class files. Thus, you need to remove your name and date.
Second, you need to write description for new added class files.

For the reference, I just reviewed the patch simply. I'll review it continuously without above comments.

@blrunner
Copy link
Contributor

Could you trigger this for Travis CI build?

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