New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DRILL-6548: IllegalStateException: Unexpected EMIT outcome received i… #1352

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@sohami
Contributor

sohami commented Jun 29, 2018

…n buildSchema phase

@sohami

This comment has been minimized.

Contributor

sohami commented Jun 29, 2018

@Ben-Zvi - Can you please review this ?

Issue was in BuildSchema phase TopN was receiving NONE outcome from upstream and there was a missing return statement in switch block which leads to NONE handling as EMIT handling and hence the exception.

@Ben-Zvi

+1 We overlooked that in the original PR .... (just checked)

vvysotskyi added a commit to vvysotskyi/drill that referenced this pull request Jul 1, 2018

@asfgit asfgit closed this in bf40c5c Jul 2, 2018

okalinin added a commit to okalinin/drill that referenced this pull request Jul 9, 2018

# This is a combination of 3 commits.
# The first commit's message is:
# This is a combination of 3 commits.
# The first commit's message is:
# This is a combination of 3 commits.
# The first commit's message is:
DRILL-5797: use Parquet new reader more often

# The 2nd commit message will be skipped:

#	DRILL-4020: The not-equal operator returns incorrect results when used on the HBase row key
#
#	- Added a condition that checks if the filter to the scan specification doesn't have NOT_EQUAL operator
#	- Added testFilterPushDownRowKeyNotEqual() to TestHBaseFilterPushDown
#
#	closes apache#309

# The 3rd commit message will be skipped:

#	DRILL-4580: Support for exporting storage plugin configurations
#
#	closes apache#1350

# The 2nd commit message will be skipped:

#	DRILL-6548: IllegalStateException: Unexpected EMIT outcome received in buildSchema phase
#
#	closes apache#1352

# The 3rd commit message will be skipped:

#	DRILL-6554: Minor code improvements in parquet statistics handling
#
#	closes apache#1349

# The 2nd commit message will be skipped:

#	DRILL-6530: JVM crash with a query involving multiple json files with one file having a schema change of one column from string to list
#
#	This closes apache#1343

# The 3rd commit message will be skipped:

#	DRILL-6561: Lateral excluding the columns from output container provided by projection push into rules
#
#	This closes apache#1356

okalinin added a commit to okalinin/drill that referenced this pull request Jul 9, 2018

# This is a combination of 3 commits.
# The first commit's message is:
# This is a combination of 3 commits.
# The first commit's message is:
# This is a combination of 3 commits.
# The first commit's message is:
DRILL-5797: use Parquet new reader more often

# The 2nd commit message will be skipped:

#	DRILL-4020: The not-equal operator returns incorrect results when used on the HBase row key
#
#	- Added a condition that checks if the filter to the scan specification doesn't have NOT_EQUAL operator
#	- Added testFilterPushDownRowKeyNotEqual() to TestHBaseFilterPushDown
#
#	closes apache#309

# The 3rd commit message will be skipped:

#	DRILL-4580: Support for exporting storage plugin configurations
#
#	closes apache#1350

# The 2nd commit message will be skipped:

#	DRILL-6548: IllegalStateException: Unexpected EMIT outcome received in buildSchema phase
#
#	closes apache#1352

# The 3rd commit message will be skipped:

#	DRILL-6554: Minor code improvements in parquet statistics handling
#
#	closes apache#1349

# The 2nd commit message will be skipped:

#	DRILL-6530: JVM crash with a query involving multiple json files with one file having a schema change of one column from string to list
#
#	This closes apache#1343

# The 3rd commit message will be skipped:

#	DRILL-6561: Lateral excluding the columns from output container provided by projection push into rules
#
#	This closes apache#1356

okalinin added a commit to okalinin/drill that referenced this pull request Jul 9, 2018

# This is a combination of 3 commits.
# The first commit's message is:
# This is a combination of 3 commits.
# The first commit's message is:
# This is a combination of 3 commits.
# The first commit's message is:
DRILL-5797: use Parquet new reader more often

# The 2nd commit message will be skipped:

#	DRILL-4020: The not-equal operator returns incorrect results when used on the HBase row key
#
#	- Added a condition that checks if the filter to the scan specification doesn't have NOT_EQUAL operator
#	- Added testFilterPushDownRowKeyNotEqual() to TestHBaseFilterPushDown
#
#	closes apache#309

# The 3rd commit message will be skipped:

#	DRILL-4580: Support for exporting storage plugin configurations
#
#	closes apache#1350

# The 2nd commit message will be skipped:

#	DRILL-6548: IllegalStateException: Unexpected EMIT outcome received in buildSchema phase
#
#	closes apache#1352

# The 3rd commit message will be skipped:

#	DRILL-6554: Minor code improvements in parquet statistics handling
#
#	closes apache#1349

# The 2nd commit message will be skipped:

#	DRILL-6530: JVM crash with a query involving multiple json files with one file having a schema change of one column from string to list
#
#	This closes apache#1343

# The 3rd commit message will be skipped:

#	DRILL-6561: Lateral excluding the columns from output container provided by projection push into rules
#
#	This closes apache#1356
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment