Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rebase linked list branch #374

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Rebase linked list branch #374

wants to merge 9 commits into from

Conversation

mxstbr
Copy link

@mxstbr mxstbr commented Mar 12, 2018

This takes the linkedlist branch from #135 and merges the latest master into it. We're running into connection errors, and we have tons of changefeeds, so I'm suspecting this could be the solution.

I'm going to try and get all the tests to pass again.

@mxstbr mxstbr mentioned this pull request Mar 12, 2018
@mxstbr
Copy link
Author

mxstbr commented Mar 12, 2018

A bunch of tests are failing locally, digging in.

EDIT: Basically all queries time out, they never return anything. Trying to figure out why.

@mxstbr
Copy link
Author

mxstbr commented Mar 12, 2018

Turns out a simple null => false change fixed most tests! (ccd4ae5) 2 test failures remaining, 794 passing!

@mxstbr
Copy link
Author

mxstbr commented Mar 12, 2018

The only two tests failing are the ones for backtraces, are those meant to fail?

@mxstbr
Copy link
Author

mxstbr commented Mar 12, 2018

All backtrace tests are fixed! Unsure why those were failing since the backtraces were correct, just slightly different in formatting.

Anyways, all the tests pass now! 🎉 @neumino I'm going to try this branch in prod I think, let's see if it fixes our errors.

mxstbr added a commit to withspectrum/spectrum that referenced this pull request Mar 12, 2018
mxstbr added a commit to withspectrum/spectrum that referenced this pull request Mar 12, 2018
@mxstbr
Copy link
Author

mxstbr commented Mar 12, 2018

Huh, some backtrace tests still fail on wercker even though they pass locally? 🤔

@mxstbr
Copy link
Author

mxstbr commented Mar 12, 2018

Even with this branch we're getting changefeeds dropping out:

ReqlServerError: The connection was closed before the query could be completed for:
r.table("usersNotifications").changes({
  includeInitial: false
}).filter(r.row("old_val").eq(null).and(r.row("new_val").eq(null).not()).or(r.row("old_val")("entityAddedAt").ne(r.row("new_val")("entityAddedAt"))))("new_val")
    .filter({
        userId: "gVk5mYwccUOEKiN5vtOouqroGKo1"
    }).eqJoin("notificationId", r.table("notifications")).without({
        left: ["notificationId", "createdAt", "id", "entityAddedAt"]
    }).zip().filter(function(var_278) {
        return var_278("context")("type").ne("DIRECT_MESSAGE_THREAD")
    })

And also the server crashing due to:

Unhandled rejection { ReqlDriverError: The connection was closed by the other party.
    at Connection._send (/home/nowuser/src/node_modules/rethinkdbdash/lib/connection.js:734:15)
    at /home/nowuser/src/node_modules/rethinkdbdash/lib/term.js:202:22
    at tryCatcher (/home/nowuser/src/node_modules/bluebird/js/release/util.js:16:23)
    at Promise._settlePromiseFromHandler (/home/nowuser/src/node_modules/bluebird/js/release/promise.js:512:31)
    at Promise._settlePromise (/home/nowuser/src/node_modules/bluebird/js/release/promise.js:569:18)
    at Promise._settlePromiseCtx (/home/nowuser/src/node_modules/bluebird/js/release/promise.js:606:10)
    at Async._drainQueue (/home/nowuser/src/node_modules/bluebird/js/release/async.js:138:12)
    at Async._drainQueues (/home/nowuser/src/node_modules/bluebird/js/release/async.js:143:10)
    at Immediate.Async.drainQueues (/home/nowuser/src/node_modules/bluebird/js/release/async.js:17:14)
    at runCallback (timers.js:789:20)
    at tryOnImmediate (timers.js:751:5)
    at processImmediate [as _immediateCallback] (timers.js:722:5)
  msg: 'The connection was closed by the other party',
  message: 'The connection was closed by the other party.',
  isOperational: true }

@thelinuxlich
Copy link

@nikhilrayaprolu
Copy link

@mxstbr @thelinuxlich
What is the final solution for this issue of changefeeds dropping?

@thelinuxlich
Copy link

thelinuxlich commented Feb 24, 2022

@nikhilrayaprolu replace your database with something supported, seriously, who would trust their primary database to a project that hasn't received updates in 5 years?

@nikhilrayaprolu
Copy link

nikhilrayaprolu commented Feb 25, 2022 via email

@mxstbr
Copy link
Author

mxstbr commented Feb 25, 2022

What is the final solution for this issue of changefeeds dropping?

Not using RethinkDB anymore 🙈

I am wondering from an engineering standpoint, how this issue was solved finally.

It wasn't.

@nikhilrayaprolu
Copy link

@mxstbr I really liked the source code of spectrum, it has all features needed for a good online community. The only major issue I found in that is the choice of rethinkDB. Now an option for me is to see if I could convert the rethinkDB to MongoDB and use Apollo PubSub for real-time features (I think this is not a priority now but need to do this before I start facing scalability issues).

Since I am planning to host an online clubs for k12 kids with some customizations (gamification) and video rooms for them to collaborate and learn together, I cannot think of going with a solution like circle.so

@thelinuxlich
Copy link

You don't need mongodb for realtime features in 2022, graphql subscriptions over sse is a good choice

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants