-
Notifications
You must be signed in to change notification settings - Fork 35
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
Some issues I noticed with latest commits #52
Comments
Did you do a I added a ticket to truncate file lists since I am working on updating the system as a whole. Where does that invalid character come from? |
Yes.
Unsure! It appears exactly like that in the database. � |
Could it be that his server blocks access to UDP or the tracker? would you get some error if so? it seems 90% of his torrents have a status of NULL in tacker updated in mysql |
Maybe. Can you set Add a line to src/tracker.js on line 58 of |
Just tested with portquiz to make sure port 6881 isn't blocked: $ curl portquiz.net:6881 $ wget -qO- portquiz.net:6881
Yes it is.
I see in the log for scraper now: |
The logs show |
Ahhaaaa. So made some changes to see if maybe I am being rate limited or something:
And now it works! I am seeing records like this now:
What a relief! Q: Is there a way to just run the SE:LE updater without scraping more torrents? I tried only running loader.js but that doesn't seem to work. |
Q: Is there a way to just run the SE:LE updater without scraping more torrents? I tried only running loader.js but that doesn't seem to work I just changed the bind ip address in config/index.js from 0.0.0.0 to 1.1.1.1 to disable getting new torrents |
tyvm =] |
I am splitting up the tracker and scraper in the current working branch which will allow this |
im also now struggling with the tracker.
|
After checking it looks like the scraper seems to be scraping 75 every 1 minute give or take. Could it be that the tracker.js is querying MySQL for torrents with seeders info less than X age and this query is taking time and the tracker has to wait for this before it can update? @Prefinem |
i can confirm this by checking the process list of mysql. and also running the command myself. Showing rows 0 - 74 (75 total, Query took 96.7467 seconds.) |
maybe i need to tweak mysql lol |
This is something I just ran into with reworking things. I think I am going
to have to drop MySQL completely since it’s too slow.
…On Thu, Feb 7, 2019 at 18:02 ash121121 ***@***.***> wrote:
i can confirm this by checking the process list of mysql. and also running
the command myself.
Showing rows 0 - 74 (75 total, Query took 96.7467 seconds.)
select * from torrents where trackerUpdated is null limit 75
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#52 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACnvZhjB5Y-nsVP0S8PC4h6Wj-b0a1knks5vLL6NgaJpZM4amw9r>
.
|
i found tweaking the InnoDB pool size took the query time down to 1 second |
What was the tweak?
…On Thu, Feb 7, 2019 at 18:26 ash121121 ***@***.***> wrote:
i found tweaking the InnoDB pool size took the query time down to 1 second
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#52 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACnvZsgouFigBYECT-wZ-xLPkkkgzuNxks5vLMQ1gaJpZM4amw9r>
.
|
i dont know what all the settings do but i tweaked inside # InnoDB Settings.
|
ofcourse you will need to tweak depending on your ram |
@Prefinem using this query cuts time down to 0.5 seconds. instead of selecting all colloms we select the table we only need. can you change this in tracker?
|
You need the infohash from the table, but I can make the changes. |
oh i see my bad |
No worries. I added the correct select in the query. Hopefully it will
speed it up
…On Thu, Feb 7, 2019 at 7:29 PM ash121121 ***@***.***> wrote:
oh i see my bad
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#52 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACnvZrWw4Hl5gGXUXU60eEvE768eLoqaks5vLNLqgaJpZM4amw9r>
.
|
Now its super speed :) |
shall we do this also for getting records in loader? |
vs
|
or do we need to query all collunms in loader? |
In the loader, yes
…On Thu, Feb 7, 2019 at 19:42 ash121121 ***@***.***> wrote:
or do we need to query all collunms in loader?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#52 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACnvZrqYxbXsvcDdOuOciuieFPcbGWVqks5vLNYBgaJpZM4amw9r>
.
|
if you choose to drop mysql. do you have an alternative idea? |
Redid for fast storage and just elasticsearch for the data
…On Thu, Feb 7, 2019 at 19:53 ash121121 ***@***.***> wrote:
if you choose to drop mysql. do you have an alternative idea?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#52 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACnvZobDblm4uaKkIrToJGupNbQlMFgvks5vLNiIgaJpZM4amw9r>
.
|
:) nice! BTW i was checking how many outdated records i have and looks like the updater cannot keep up.You can see it increasing on each query.
|
So lastnight i stopped crawling new torrents and let it update torrents with trackerupdate = null . |
it appears overnight its not even querying the database to update trackers... and i havent changed anything.so strange on hetzner server and digital ocean. update.... seems opensurfer not working same for milez. |
So glad to see @Prefinem working on this again =]
Total Torrents: 785004
Torrents without Tracker: 781921
Torrents not in Search: 361
I did this to test (
ALTER TABLE torrents MODIFY `files` MEDIUMTEXT;
) and it seems no more broken file lists but this opens us up to 16mb max record sizes on this field. Might be worth it to truncate these instead in parser.js but I'm not sure the best approach :/0xFFFD, �
- and a few others...but I'll keep trying. I haven't tested but I think this should work on all unexpected characters:dataStr.replace(/[\u{0080}-\u{FFFF}]/gu,"");
The text was updated successfully, but these errors were encountered: