experiments
/
cloudtube_fork
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Remove logging from background refresh 

I'm happy enough with the stability of all code paths to not need
extensive logging any more.
This commit is contained in:
Cadence Ember 2021-08-18 12:17:12 +12:00
parent 57e4507bff
commit 83c34f75d9
No known key found for this signature in database
GPG Key ID: BC1C2C61CF521B17
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
background/feed-update.js
View File

@ -87,12 +87,12 @@ class Refresher {
}) })
// update channel refreshed // update channel refreshed
prepared.channel_refreshed_update.run(Date.now(), ucid) prepared.channel_refreshed_update.run(Date.now(), ucid)
console.log(`updated ${root.length} videos for channel ${ucid}`) // console.log(`updated ${root.length} videos for channel ${ucid}`)
} else if (root.identifier === "PUBLISHED_DATES_NOT_PROVIDED") { } else if (root.identifier === "PUBLISHED_DATES_NOT_PROVIDED") {
return [] // nothing we can do. skip this iteration. return [] // nothing we can do. skip this iteration.
} else if (root.identifier === "NOT_FOUND") { } else if (root.identifier === "NOT_FOUND") {
// the channel does not exist. we should unsubscribe all users so we don't try again. // the channel does not exist. we should unsubscribe all users so we don't try again.
console.log(`channel ${ucid} does not exist, unsubscribing all users`) // console.log(`channel ${ucid} does not exist, unsubscribing all users`)
prepared.unsubscribe_all_from_channel.run(ucid) prepared.unsubscribe_all_from_channel.run(ucid)
} else { } else {
throw new Error(root.error) throw new Error(root.error)
@ -105,7 +105,7 @@ class Refresher {
const timeSinceLastLoop = Date.now() - this.refreshQueue.lastLoadTime const timeSinceLastLoop = Date.now() - this.refreshQueue.lastLoadTime
if (timeSinceLastLoop < constants.caching.subscriptions_refresh_loop_min) { if (timeSinceLastLoop < constants.caching.subscriptions_refresh_loop_min) {
const timeToWait = constants.caching.subscriptions_refresh_loop_min - timeSinceLastLoop const timeToWait = constants.caching.subscriptions_refresh_loop_min - timeSinceLastLoop
console.log(`waiting ${timeToWait} before next loop`) // console.log(`waiting ${timeToWait} before next loop`)
this.state = this.sym.WAITING this.state = this.sym.WAITING
this.waitingTimeout = setTimeout(() => this.next(), timeToWait) this.waitingTimeout = setTimeout(() => this.next(), timeToWait)
return return
@ -121,7 +121,7 @@ class Refresher {
// Problems related to fetching from the instance? // Problems related to fetching from the instance?
// All we can do is retry later. // All we can do is retry later.
// However, skip this channel this time in case the problem will occur every time. // However, skip this channel this time in case the problem will occur every time.
console.error(error) console.error("Error in background refresh:\n", error)
setTimeout(() => { setTimeout(() => {
this.next() this.next()
}, 10e3) }, 10e3)