You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by GitBox <gi...@apache.org> on 2022/03/30 13:52:50 UTC
[GitHub] [calcite-avatica-go] liguozhong opened a new pull request #56: Fix bug：panic msg: runtime error: index out of range [0] with length 0
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/v5@v5.1.1-0.20220330132417-bb5de288ce8c/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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [calcite-avatica-go] liguozhong commented on pull request #56: Fix bug：panic msg: runtime error: index out of range [0] with length 0
Posted by GitBox <gi...@apache.org>.
liguozhong commented on pull request #56:
URL: https://github.com/apache/calcite-avatica-go/pull/56#issuecomment-1083205267
```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).Next(0x4cc4b7, {0x4ca3a78, 0x1fc3d9f, 0xc05912c140})
/Users/fuling/go/pkg/mod/github.com/liguozhong/calcite-avatica-go/v5@v5.1.1-0.20220330134131-bddb6756a60d/rows.go:83 +0x536
database/sql.(*Rows).nextLocked(0xc01499f000)
/usr/local/go/src/database/sql/sql.go:2967 +0x111
database/sql.(*Rows).Next.func1()
/usr/local/go/src/database/sql/sql.go:2945 +0x2f
database/sql.withLock({0x32a6008, 0xc01499f030}, 0xc0092536f0)
/usr/local/go/src/database/sql/sql.go:3396 +0x8c
database/sql.(*Rows).Next(0xc01499f000)
/usr/local/go/src/database/sql/sql.go:2944 +0x6f
github.com/grafana/loki/pkg/storage/chunk/avatica.(*StorageClient).query(0xc00120e680, {0x32d98a8, 0xc0035e6150}, {{0xc005966a40, 0x1e}, {0xc003c36ff0, 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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [calcite-avatica-go] liguozhong closed pull request #56: Fix bug：panic msg: runtime error: index out of range [0] with length 0
Posted by GitBox <gi...@apache.org>.
liguozhong closed pull request #56:
URL: https://github.com/apache/calcite-avatica-go/pull/56
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [calcite-avatica-go] F21 closed pull request #56: [CALCITE-5072] Index out of range when calling rows.Next()
Posted by GitBox <gi...@apache.org>.
F21 closed pull request #56:
URL: https://github.com/apache/calcite-avatica-go/pull/56
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [calcite-avatica-go] F21 commented on pull request #56: [CALCITE-5072] Index out of range when calling rows.Next()
Posted by GitBox <gi...@apache.org>.
F21 commented on pull request #56:
URL: https://github.com/apache/calcite-avatica-go/pull/56#issuecomment-1084094003
Fixed in 945cf887766c83b1af8e9aa2f80c2803e6c65626.
Thanks for your help!
--
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: commits-unsubscribe@calcite.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org