-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
go.mod
Outdated
@@ -11,15 +11,18 @@ replace google.golang.org/genproto => google.golang.org/genproto v0.0.0-20180514 | |||
go 1.12 | |||
|
|||
require ( | |||
github.com/apache/arrow/go/arrow v0.0.0-20190716210558-5f564424c71c | |||
github.com/golang/protobuf v1.2.0 // indirect | |||
github.com/apache/arrow/go/arrow v0.0.0-20191004105443-1e2cf1f95df0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is latest here going to work with the frontend? Before we were having issues if it wasn't at 0.14.1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
replace github.com/apache/arrow/go/arrow => github.com/apache/arrow/go/arrow v0.0.0-20190716210558-5f564424c71c
makes the UI work again for me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, this is my lack of Go module skills. I'll update it!
Fixes #10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we are good to merge once the arrow dep in mod is fixed and the refId assignment is uncommented. Thanks, this is great! One less duplicated thing around all this.
pkg/gelpoc/service.go
Outdated
for _, val := range results.Values { | ||
df := val.AsDataFrame() | ||
df.RefID = refID | ||
//df.RefID = refID |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is needed for the filteredFrames (hidden) functionality to work.
39fc5fa
to
2a1e8b9
Compare
No description provided.