Skip to content
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

Don't use fixed hash local exchange for single stream table writers #2388

Merged
merged 1 commit into from Jan 7, 2020

Conversation

@sopel39
Copy link
Member

sopel39 commented Jan 2, 2020

LocalExecutionPlanner.Visitor#visitTableWriter sets
setDriverInstanceCount to 1 when getPartitioningScheme is present.
This changes makes planner use gather local exchange in that case.

LocalExecutionPlanner.Visitor#visitTableWriter sets
setDriverInstanceCount to 1 when getPartitioningScheme is present.
This changes makes planner use gather local exchange in that case.
@sopel39 sopel39 requested review from martint and findepi Jan 2, 2020
@cla-bot cla-bot bot added the cla-signed label Jan 2, 2020
@findepi findepi requested a review from electrum Jan 3, 2020
@electrum electrum requested review from dain and removed request for electrum Jan 3, 2020
@dain
dain approved these changes Jan 7, 2020
@sopel39 sopel39 merged commit 76abeb3 into prestosql:master Jan 7, 2020
34 checks passed
34 checks passed
maven-checks
Details
error-prone-checks
Details
web-ui-checks
Details
hive-tests (config-empty)
Details
hive-tests (config-hdp3)
Details
kudu-tests
Details
test-other-modules
Details
test-modules (presto-main)
Details
test-modules (presto-tests) test-modules (presto-tests)
Details
test-modules (presto-tests -P ci-only)
Details
test-modules (presto-raptor-legacy)
Details
test-modules (presto-accumulo)
Details
test-modules (presto-cassandra)
Details
test-modules (presto-hive,presto-orc)
Details
test-modules (presto-hive,presto-parquet -P test-parquet)
Details
test-modules (presto-mongodb,presto-kafka,presto-elasticsearch)
Details
test-modules (presto-redis)
Details
test-modules (presto-sqlserver,presto-postgresql,presto-mysql)
Details
test-modules (presto-phoenix,presto-iceberg)
Details
product-tests (config-empty, suite-1)
Details
product-tests (config-empty, suite-2)
Details
product-tests (config-empty, suite-3)
Details
product-tests (config-empty, suite-5)
Details
product-tests (config-empty, suite-6-non-generic)
Details
product-tests (config-empty, suite-7-non-generic)
Details
product-tests (config-hdp3, suite-1)
Details
product-tests (config-hdp3, suite-2)
Details
product-tests (config-hdp3, suite-3)
Details
product-tests (config-hdp3, suite-5)
Details
product-tests (config-cdh5, suite-1)
Details
product-tests (config-cdh5, suite-2)
Details
product-tests (config-cdh5, suite-3)
Details
product-tests (config-cdh5, suite-5)
Details
verification/cla-signed
Details
@sopel39 sopel39 deleted the starburstdata:ks/use_gather_writer branch Jan 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
You can’t perform that action at this time.