Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Apr 07 15:52
    GaikwadPratik closed #493
  • Apr 07 15:14
    Gerrit91 edited #500
  • Apr 07 15:09
    Gerrit91 edited #500
  • Apr 07 15:05
    Gerrit91 edited #500
  • Apr 07 15:02
    Gerrit91 edited #500
  • Apr 07 14:58
    Gerrit91 edited #500
  • Apr 07 14:52
    Gerrit91 edited #500
  • Apr 07 14:42
    Gerrit91 commented #493
  • Apr 07 13:45
    Gerrit91 edited #500
  • Apr 07 13:44
    Gerrit91 edited #500
  • Apr 07 13:37
    Gerrit91 opened #500
  • Sep 26 2021 22:12
    CodyDWJones edited #499
  • Sep 26 2021 22:12
    CodyDWJones opened #499
  • Sep 26 2021 20:07
    CodyDWJones closed #498
  • Sep 24 2021 17:24
    CodyDWJones commented #497
  • Sep 24 2021 17:06
    CodyDWJones closed #497
  • Sep 23 2021 21:42
    CodyDWJones commented #498
  • Sep 23 2021 14:35
    CMogilko commented #498
  • Sep 23 2021 13:49

    CMogilko on master

    Upgrade to Python 3.8 (#496) *… (compare)

  • Sep 23 2021 13:49
    CMogilko closed #496
Daniel Cannon
@dancannon
Hi @prophittcorey just having a look now
Hmm i wonder if the value of GeneratedKeys is correct, could you try running your code and log that value and see what is logged when your session is empty?
And just to double check, you are checking the error returned by both Run and One?
Daniel Cannon
@dancannon
Ok great thanks for checking
@prophittcorey would it be possible for you to post the entire code so I can run some more detailed tests? If you are not comfortable posting here you could maybe email it?
Great thanks
Daniel Cannon
@dancannon
@prophittcorey Ok I think I have found the cause of the issue
There seems to be an issue when you unmarshal the time into the CreatedAt term
I dont think so no
For now you can fix your code by changing the CreatedAt field to time.Time and setting it with time.Now() instead of db.Now()
Just out of curiosity why did you decide to use the rethinkdb time functions?
Yeah even though I think there is an issue with the driver not handling this properly (it should probably error since you cant convert a time back to a gorethink Term) I think using the Go time package is the best thing for you to do
No worries, let me know if you have any other questions
Daniel Cannon
@dancannon
@prophittcorey Oh just noticed you were not actually checking the error returned by One. It looks like it was actually returning an error
Daniel Cannon
@dancannon
@prophittcorey You just need err = cursor.One(&session) instead of just cursor.One(&session)
Suyash Katiyar
@skatiyar
@dancannon is current release of driver compatible with rethinkdb 2.2??
Daniel Cannon
@dancannon
@SKatiyar I have not yet update the driver to support the new features but it should be compatible . Let me know if you have any issues
Suyash Katiyar
@skatiyar
@dancannon was thinking of putting new changefeed feature in driver
Daniel Cannon
@dancannon
If you have the time I would really appreciate it.
Suyash Katiyar
@skatiyar
cool should open a issue first??
Daniel Cannon
@dancannon
Sure feel free
Suyash Katiyar
@skatiyar
and any particular point from where i can start??
Daniel Cannon
@dancannon
I will add it to the 1.2 milestone
I havnt looked into what needs to be changed yet but I imagine it will require changing the options struct for the Changes term and creating some new tests for the new response structure
Suyash Katiyar
@skatiyar
adding new option to struct, apart from that do i need to touch any other functionality??
Daniel Cannon
@dancannon
I have no idea sorry! Usually the first thing I do is look at how one of the official drivers has implemented the change
Suyash Katiyar
@skatiyar
cool, will do that. Will ask you if I run into any problems.
Daniel Cannon
@dancannon
Yeah feel free, I should be online for a while
Suyash Katiyar
@skatiyar
how can we see output while running tests??
go test -v doesn’t seem to work
fazal
@faxal
with Log or Logf
Suyash Katiyar
@skatiyar
@dancannon to add new command values do we have to get proto file from rethinkdb repo??
Daniel Cannon
@dancannon
@SKatiyar Yes the proto file needs to be updated
Regarding the test output try using the flag -gocheck.vv
Suyash Katiyar
@skatiyar
@dancannon driver doesn't seem to have connection event implemented. Should I look into implementing them??
Daniel Cannon
@dancannon
@SKatiyar What do you mean by "connection event"?
Suyash Katiyar
@skatiyar
http://rethinkdb.com/api/javascript/#event_emitter
Daniel Cannon
@dancannon
Hmm, can you think of a use case for this?
My understanding is that this is provided by the JS driver as it is a feature of the JS connection itself
Suyash Katiyar
@skatiyar
as of now only reconnect seems only one
Daniel Cannon
@dancannon
I dont think it is in any other drivers
In gorethink this tends to be handled internally
Especially since GoRethink has a host and connection pool so the driver user shouldnt really need to care about reconnections
Suyash Katiyar
@skatiyar
ok cool was wondering if its driver specific or more of a rethink thing
Daniel Cannon
@dancannon
If you think it would be useful + know how to implement it then go for it. I am just not sure if we need it
Suyash Katiyar
@skatiyar
for now lets punt it, if needed will do it.
what drivers do you take for reference usually??
Daniel Cannon
@dancannon
Python + rethinkdbdash
Daniel Cannon
@dancannon
@SKatiyar have you started work on .Server()?
Daniel Cannon
@dancannon
I had a go at it as well and changed the cluster connection code to use a hostpool. Let me know what you think @SKatiyar https://github.com/dancannon/gorethink/compare/develop...feature/server-info
If you have already worked on the Server function I would like to see how you solved it