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

[Go][Flight] Verify soruce nightly jobs are failing sqlite_server_test.go TestSqliteServer/TestCommandGetTablesWithIncludedSchemas #32757

Closed
asfimport opened this issue Aug 22, 2022 · 4 comments

Comments

@asfimport
Copy link
Collaborator

Some of our verify-source nightly jobs as:
verify-rc-source-go-linux-conda-latest-amd64
verify-rc-source-go-macos-amd64
verify-rc-source-integration-linux-conda-latest-amd64
verify-rc-source-integration-macos-amd64
verify-rc-source-integration-macos-conda-amd64

failed with the following error:

--- FAIL: TestSqliteServer (0.08s)
    --- FAIL: TestSqliteServer/TestCommandGetTablesWithIncludedSchemas (0.00s)
        sqlite_server_test.go:324: 
                Error Trace:    /Users/runner/work/crossbow/crossbow/arrow/go/arrow/flight/flightsql/sqlite_server_test.go:324
                Error:          Should be true
                Test:           TestSqliteServer/TestCommandGetTablesWithIncludedSchemas
                Messages:       expected: record:
                                  schema:
                                  fields: 5
                                    - catalog_name: type=utf8, nullable
                                    - db_schema_name: type=utf8, nullable
                                    - table_name: type=utf8
                                    - table_type: type=utf8
                                    - table_schema: type=binary
                                  rows: 1
                                  col[0][catalog_name]: [(null)]
                                  col[1][db_schema_name]: [(null)]
                                  col[2][table_name]: ["intTable"]
                                  col[3][table_type]: ["table"]
                                  col[4][table_schema]: ["\xff\xff\xff\xff\xa8\x05\x00\x00\x10\x00\x00\x00\x00\x00\n\x00\f\x00\n\x00\t\x00\x04\x00\n\x00\x00\x00\x10\x00\x00\x00\x00\x01\x04\x00\b\x00\b\x00\x00\x00\x04\x00\b\x00\x00\x00\x04\x00\x00\x00\x04\x00\x00\x00\f\x04\x00\x00\xd0\x02\x00\x00l\x01\x00\x00\x04\x00\x00\x00\x16\xfc\xff\xff\x14\x00\x00\x00<\x01\x00\x00<\x01\x00\x00\x00\x00\x00\x02@\x01\x00\x00\x05\x00\x00\x00\xf8\x00\x00\x00\xbc\x00\x00\x00|\x00\x00\x00<\x00\x00\x00\x04\x00\x00\x00 \xfb\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x02\x00\x00\x0010\x00\x00\x1a\x00\x00\x00ARROW:FLIGHT:SQL:PRECISION\x00\x00T\xfb\xff\xff\b\x00\x00\x00\x14\x00\x00\x00\b\x00\x00\x00intTable\x00\x00\x00\x00\x1b\x00\x00\x00ARROW:FLIGHT:SQL:TABLE_NAME\x00\x90\xfb\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x01\x00\x00\x000\x00\x00\x00\"\x00\x00\x00ARROW:FLIGHT:SQL:IS_AUTO_INCREMENT\x00\x00\xcc\xfb\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x01\x00\x00\x000\x00\x00\x00\x1d\x00\x00\x00ARROW:FLIGHT:SQL:IS_READ_ONLY\x00\x00\x00\x04\xfc\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x02\x00\x00\x0015\x00\x00\x16\x00\x00\x00ARROW:FLIGHT:SQL:SCALE\x00\x00\x00\x00\x00\x00\xfc\xfb\xff\xff\x00\x00\x00\x01@\x00\x00\x00\t\x00\x00\x00foreignId\x00\x00\x00z\xfd\xff\xff\x14\x00\x00\x00<\x01\x00\x00<\x01\x00\x00\x00\x00\x00\x02@\x01\x00\x00\x05\x00\x00\x00\xf8\x00\x00\x00\xbc\x00\x00\x00|\x00\x00\x00<\x00\x00\x00\x04\x00\x00\x00\x84\xfc\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x02\x00\x00\x0010\x00\x00\x1a\x00\x00\x00ARROW:FLIGHT:SQL:PRECISION\x00\x00\xb8\xfc\xff\xff\b\x00\x00\x00\x14\x00\x00\x00\b\x00\x00\x00intTable\x00\x00\x00\x00\x1b\x00\x00\x00ARROW:FLIGHT:SQL:TABLE_NAME\x00\xf4\xfc\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x01\x00\x00\x000\x00\x00\x00\"\x00\x00\x00ARROW:FLIGHT:SQL:IS_AUTO_INCREMENT\x00\x000\xfd\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x01\x00\x00\x000\x00\x00\x00\x1d\x00\x00\x00ARROW:FLIGHT:SQL:IS_READ_ONLY\x00\x00\x00h\xfd\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x02\x00\x00\x0015\x00\x00\x16\x00\x00\x00ARROW:FLIGHT:SQL:SCALE\x00\x00\x00\x00\x00\x00`\xfd\xff\xff\x00\x00\x00\x01@\x00\x00\x00\x05\x00\x00\x00value\x00\x00\x00\xda\xfe\xff\xff\x14\x00\x00\x00\x04\x01\x00\x00\b\x01\x00\x00\x00\x00\x00\x05\x04\x01\x00\x00\x04\x00\x00\x00\xc0\x00\x00\x00\x84\x00\x00\x00D\x00\x00\x00\x04\x00\x00\x00\xe0\xfd\xff\xff\b\x00\x00\x00\x14\x00\x00\x00\b\x00\x00\x00intTable\x00\x00\x00\x00\x1b\x00\x00\x00ARROW:FLIGHT:SQL:TABLE_NAME\x00\x1c\xfe\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x01\x00\x00\x000\x00\x00\x00\"\x00\x00\x00ARROW:FLIGHT:SQL:IS_AUTO_INCREMENT\x00\x00X\xfe\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x01\x00\x00\x000\x00\x00\x00\x1d\x00\x00\x00ARROW:FLIGHT:SQL:IS_READ_ONLY\x00\x00\x00\x90\xfe\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x02\x00\x00\x0015\x00\x00\x16\x00\x00\x00ARROW:FLIGHT:SQL:SCALE\x00\x00\x00\x00\x00\x00\x04\x00\x04\x00\x04\x00\x00\x00\a\x00\x00\x00keyName\x00\x00\x00\x12\x00\x18\x00\x14\x00\x00\x00\x13\x00\f\x00\x00\x00\b\x00\x04\x00\x12\x00\x00\x00\x14\x00\x00\x00D\x01\x00\x00L\x01\x00\x00\x00\x00\x00\x02P\x01\x00\x00\x05\x00\x00\x00\x00\x01\x00\x00\xbc\x00\x00\x00|\x00\x00\x00<\x00\x00\x00\x04\x00\x00\x00\x1c\xff\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x02\x00\x00\x0010\x00\x00\x1a\x00\x00\x00ARROW:FLIGHT:SQL:PRECISION\x00\x00P\xff\xff\xff\b\x00\x00\x00\x14\x00\x00\x00\b\x00\x00\x00intTable\x00\x00\x00\x00\x1b\x00\x00\x00ARROW:FLIGHT:SQL:TABLE_NAME\x00\x8c\xff\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x01\x00\x00\x000\x00\x00\x00\"\x00\x00\x00ARROW:FLIGHT:SQL:IS_AUTO_INCREMENT\x00\x00\xc8\xff\xff\xff\b\x00\x00\x00\f\x00\x00\x00\x01\x00\x00\x000\x00\x00\x00\x1d\x00\x00\x00ARROW:FLIGHT:SQL:IS_READ_ONLY\x00\x00\x00\b\x00\f\x00\b\x00\x04\x00\b\x00\x00\x00\b\x00\x00\x00\f\x00\x00\x00\x02\x00\x00\x0015\x00\x00\x16\x00\x00\x00ARROW:FLIGHT:SQL:SCALE\x00\x00\x00\x00\x00\x00\b\x00\f\x00\b\x00\a\x00\b\x00\x00\x00\x00\x00\x00\x01@\x00\x00\x00\x02\x00\x00\x00id\x00\x00\xff\xff\xff\xff\x00\x00\x00\x00"]
                                
                                got: record:
                                  schema:
                                  fields: 5
                                    - catalog_name: type=utf8, nullable
                                    - db_schema_name: type=utf8, nullable
                                    - table_name: type=utf8
                                    - table_type: type=utf8
                                    - table_schema: type=binary
                                  rows: 1
                                  col[0][catalog_name]: [(null)]
                                  col[1][db_schema_name]: [(null)]
                                  col[2][table_name]: ["intTable"]
                                  col[3][table_type]: ["table"]
                                  col[4][table_schema]: ["\xff\xff\xff\xff0\x00\x00\x00\x10\x00\x00\x00\x00\x00\n\x00\f\x00\n\x00\t\x00\x04\x00\n\x00\x00\x00\x10\x00\x00\x00\x00\x01\x04\x00\b\x00\b\x00\x00\x00\x04\x00\b\x00\x00\x00\x04\x00\x00\x00\x00\x00\x00\x00\xff\xff\xff\xff\x00\x00\x00\x00"]
FAIL
FAIL    github.com/apache/arrow/go/v10/arrow/flight/flightsql    0.402s 

Reporter: Raúl Cumplido / @raulcd
Assignee: Matthew Topol / @zeroshade

PRs and other links:

Note: This issue was originally created as ARROW-17496. Please see the migration documentation for further details.

@asfimport
Copy link
Collaborator Author

Raúl Cumplido / @raulcd:
cc ~ @zeroshade  

@asfimport
Copy link
Collaborator Author

Matthew Topol / @zeroshade:
Well this is odd as hell, I can't reproduce myself. But it looks like the common thread is go1.17, which is slightly annoying given that the sqlite3 package says it supports go1.17.

I'll dig into this a bit and possibly mark this test (and the example sqlite server itself) as only build with go1.18 if I need to.

@asfimport
Copy link
Collaborator Author

Matthew Topol / @zeroshade:
@raulcd I confirmed the issue is the pragma_table_info call when run in go1.17 vs go1.18. I've put up a PR marking the sqlite flightsql example in go to only build with go1.18 (we already have github actions CI runs that run using go1.18) which should solve the issue of the nightly builds that were using go1.17 here. I'll wait for the checks to complete and then merge it.

@asfimport
Copy link
Collaborator Author

Matthew Topol / @zeroshade:
Issue resolved by pull request 13943
#13943

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants