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

Conversation

@blrunner
Copy link
Contributor

See https://issues.apache.org/jira/browse/TAJO-1952 and #846.
I recreated a pull request because of a branch deletion mistake.

blrunner added 7 commits May 16, 2016 17:08
…into TAJO-1952

Conflicts:
	tajo-core/src/main/java/org/apache/tajo/master/exec/NonForwardQueryResultFileScanner.java
	tajo-core/src/main/java/org/apache/tajo/querymaster/Repartitioner.java
	tajo-core/src/main/java/org/apache/tajo/querymaster/Stage.java
	tajo-plan/src/main/java/org/apache/tajo/plan/logical/PartitionedTableScanNode.java
…into TAJO-1952

Conflicts:
	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-storage/tajo-storage-hdfs/src/main/java/org/apache/tajo/storage/fragment/FileFragment.java
@blrunner
Copy link
Contributor Author

Not ready to review. After recent rebase, some unit tests fail. I'm trying to fix it.

@blrunner
Copy link
Contributor Author

At first, I approached to make another fragment for partitioned table. But now, the only difference between FileFragment and PartitionFileFragment is the partition key. So, I would prefer to add the variable to FileFragment instead of making PartitionFileFragment. It will decrease unnecessary modifications and will avoid side effects by PartitionFileFragment. Here are codes by above approach. https://github.com/apache/tajo/compare/master...blrunner:TAJO-1952-2?expand=1.

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.

1 participant