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

sqlccl: sf-100 import failure #20917

Closed
maddyblue opened this issue Dec 20, 2017 · 2 comments
Closed

sqlccl: sf-100 import failure #20917

maddyblue opened this issue Dec 20, 2017 · 2 comments
Labels
A-testing Testing tools and infrastructure C-investigation Further steps needed to qualify. C-label will change.

Comments

@maddyblue
Copy link
Contributor

root@:26257/> IMPORT TABLE lineitem CREATE USING 'gs://cockroach-fixtures/tpch-csv/schema/lineitem.sql'
           -> CSV DATA (
           ->     'gs://cockroach-fixtures/tpch-csv/sf-100/lineitem.tbl.1',
           ->     'gs://cockroach-fixtures/tpch-csv/sf-100/lineitem.tbl.2',
           ->     'gs://cockroach-fixtures/tpch-csv/sf-100/lineitem.tbl.3',
           ->     'gs://cockroach-fixtures/tpch-csv/sf-100/lineitem.tbl.4',
           ->     'gs://cockroach-fixtures/tpch-csv/sf-100/lineitem.tbl.5',
           ->     'gs://cockroach-fixtures/tpch-csv/sf-100/lineitem.tbl.6',
           ->     'gs://cockroach-fixtures/tpch-csv/sf-100/lineitem.tbl.7',
           ->     'gs://cockroach-fixtures/tpch-csv/sf-100/lineitem.tbl.8'
           -> ) WITH distributed, delimiter = '|', temp = 'gs://cockroach-tmp/csvtest-mjibson-100', transform_only;
pq: gs://cockroach-fixtures/tpch-csv/sf-100/lineitem.tbl.8: row 41834077: reading CSV record: stream error: stream ID 13; INTERNAL_ERROR

Nothing in the logs from around that time.

@maddyblue
Copy link
Contributor Author

This appears to be a HTTP2 error sent by GCS (see googleapis/google-cloud-go#784), so not a distsql or import error. The error occurs mid-read, so it's tricky to retry.

@knz knz added the A-testing Testing tools and infrastructure label Jul 21, 2018
@knz knz added the C-investigation Further steps needed to qualify. C-label will change. label Jul 21, 2018
@maddyblue
Copy link
Contributor Author

This has been fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testing Testing tools and infrastructure C-investigation Further steps needed to qualify. C-label will change.
Projects
None yet
Development

No branches or pull requests

2 participants