xiazcy commented on code in PR #1786:
URL: https://github.com/apache/tinkerpop/pull/1786#discussion_r951985838
##########
gremlin-go/driver/result.go:
##########
@@ -27,17 +27,17 @@ import (
// Result Struct to abstract the Result and provide functions to use it.
type Result struct {
- result interface{}
+ Result interface{}
Review Comment:
One minor item, to avoid confusion with the struct and field names being the
same, I'd suggest to change the field name to `Data` instead.
```suggestion
Data interface{}
```
So we'll have `r.Result` changed to `r.Data`.
The rest LGTM. Thanks!
--
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]
