liguozhong opened a new pull request #56: URL: https://github.com/apache/calcite-avatica-go/pull/56
fix index ount of bound panic. https://github.com/grafana/loki/pull/5692/ [new feature] Index: loki support apache calcite avatica index storage. #5692 I implemented the index storage plugin of apache avatic in the grafana loki community, and found that avatica-go would panic in the online environment. This PR attempts to fix this panic. Our online loki has been fixed. ```shell panic: runtime error: index out of range [0] with length 0 /usr/local/go/src/runtime/panic.go:1038 +0x215 github.com/apache/calcite-avatica-go/v5.(*rows).Columns(0x4cc4b7) /Users/fuling/go/pkg/mod/github.com/liguozhong/calcite-avatica-go/[email protected]/rows.go:55 +0x17b database/sql.(*Rows).nextLocked(0xc038348000) /usr/local/go/src/database/sql/sql.go:2964 +0xb0 database/sql.(*Rows).Next.func1() /usr/local/go/src/database/sql/sql.go:2945 +0x2f database/sql.withLock({0x32a6008, 0xc038348030}, 0xc004cb76f0) /usr/local/go/src/database/sql/sql.go:3396 +0x8c database/sql.(*Rows).Next(0xc038348000) /usr/local/go/src/database/sql/sql.go:2944 +0x6f github.com/grafana/loki/pkg/storage/chunk/avatica.(*StorageClient).query(0xc001024ea0, {0x32d98a8, 0xc0188ef320}, {{0xc00589f300, 0x1e}, {0xc01bc60990, 0x2b}, {0x0, 0x0, 0x0}, ...}, ...) ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
