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

TAJO-1131: Supports Inserting or Creating table into the HBase mapped table.#232

Closed
babokim wants to merge 31 commits into
apache:hbase_storagefrom
babokim:TAJO-1131
Closed

TAJO-1131: Supports Inserting or Creating table into the HBase mapped table.#232
babokim wants to merge 31 commits into
apache:hbase_storagefrom
babokim:TAJO-1131

Conversation

@babokim
Copy link
Copy Markdown
Contributor

@babokim babokim commented Nov 10, 2014

First, I apologize. The TAJO-1127 Implements HBaseStorageManager issue is also implemented in this issue together.
This issue has been implemented the following features:

Mapping HBase table to Tajo table.
Supports a composite rowkey
Supports creation multiple Region.
Scanning HBase table using Tajo selection query.
Inserting into HBase table using Tajo insertion or creation query.
Supports both of PUT and BULK.

HyoungJun Kim and others added 29 commits October 23, 2014 16:31
Implements HBaseStorageManager and HBaseScanner.
Implements non-forward query.
…sf/tajo into TAJO-1123

Conflicts:
	tajo-core/src/main/java/org/apache/tajo/engine/planner/PhysicalPlannerImpl.java
	tajo-core/src/main/java/org/apache/tajo/master/GlobalEngine.java
	tajo-core/src/main/java/org/apache/tajo/master/querymaster/Repartitioner.java
	tajo-core/src/main/java/org/apache/tajo/master/querymaster/SubQuery.java
	tajo-plan/src/main/java/org/apache/tajo/plan/util/PlannerUtil.java
…-1127

Conflicts:
	tajo-core/src/main/java/org/apache/tajo/master/NonForwardQueryResultScanner.java
	tajo-core/src/test/java/org/apache/tajo/engine/planner/TestPlannerUtil.java
	tajo-plan/src/main/java/org/apache/tajo/plan/util/PlannerUtil.java
…sf/tajo into TAJO-1127

Conflicts:
	tajo-common/src/main/java/org/apache/tajo/TajoConstants.java
	tajo-core/src/main/java/org/apache/tajo/engine/planner/PhysicalPlannerImpl.java
	tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/SeqScanExec.java
	tajo-core/src/main/java/org/apache/tajo/master/GlobalEngine.java
	tajo-core/src/main/java/org/apache/tajo/master/LazyTaskScheduler.java
	tajo-core/src/main/java/org/apache/tajo/master/NonForwardQueryResultScanner.java
	tajo-core/src/main/java/org/apache/tajo/master/querymaster/Repartitioner.java
	tajo-core/src/main/java/org/apache/tajo/master/querymaster/SubQuery.java
	tajo-storage/src/main/java/org/apache/tajo/storage/FileStorageManager.java
	tajo-storage/src/main/java/org/apache/tajo/storage/MergeScanner.java
	tajo-storage/src/main/java/org/apache/tajo/storage/StorageManager.java
… table.

Implements ColumnKey, Value mapping
… table.

Implements Insert into location using hbase
…sf/tajo into TAJO-1131

Conflicts:
	tajo-core/src/main/java/org/apache/tajo/engine/query/QueryContext.java
	tajo-plan/src/main/java/org/apache/tajo/plan/util/PlannerUtil.java
… table.

Change TupleComparator to BaseTupleComparator.
… table.

Move hbase.version property to tajo-project/pom.xml
@hyunsik
Copy link
Copy Markdown
Member

hyunsik commented Nov 11, 2014

Could you rebase it against the latest patch?

…sf/tajo into TAJO-1131

Conflicts:
	BUILDING
	tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogUtil.java
	tajo-catalog/tajo-catalog-common/src/main/proto/CatalogProtos.proto
	tajo-core/src/main/java/org/apache/tajo/master/GlobalEngine.java
	tajo-storage/pom.xml
	tajo-storage/src/main/resources/storage-default.xml
@babokim
Copy link
Copy Markdown
Contributor Author

babokim commented Nov 12, 2014

I just rebase.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems to be unused.

@hyunsik
Copy link
Copy Markdown
Member

hyunsik commented Nov 13, 2014

This is awesome work! This feature will make Tajo much better in various workloads and use cases.

I leave comments. In addition, the patch contains not few unused imports. Please remove out them.

… table.

Remove unused import or code and change variable name.
@babokim
Copy link
Copy Markdown
Contributor Author

babokim commented Nov 14, 2014

@hyunsik Thanks for your review. I pushed the fetch which is applied your comment.

@hyunsik
Copy link
Copy Markdown
Member

hyunsik commented Nov 14, 2014

Excellent! Here is my +1 for pushing the patch to hbase_storage branch.

asfgit pushed a commit that referenced this pull request Nov 16, 2014
@hyunsik
Copy link
Copy Markdown
Member

hyunsik commented Nov 17, 2014

@babokim Could you close this issue? The 'ticket close' command in git commit log seems to work in only master branch.

@babokim babokim closed this Nov 17, 2014
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