Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

High CPU when publishing files with js-ipfs #1190

Closed
ido-ran opened this issue Jan 25, 2018 · 2 comments
Closed

High CPU when publishing files with js-ipfs #1190

ido-ran opened this issue Jan 25, 2018 · 2 comments
Assignees
Labels
exp/wizard Extensive knowledge (implications, ramifications) required help wanted Seeking public contribution on this issue kind/bug A bug in existing code (including security flaws) kind/wontfix-migration-available P1 High: Likely tackled by core team if no one steps up status/ready Ready to be worked topic/perf Performance

Comments

@ido-ran
Copy link

ido-ran commented Jan 25, 2018

  • Version: js-ipfs/0.27.5
  • Platform: macOS High Sierra 10.13.2 (17C205)
  • Subsystem: Chrome Version 63.0.3239.132 (Official Build) (64-bit)

Type: Bug

Severity: Medium

Description:

I'm building a system in which js-ipfs node publish file and go-ipfs node get it, I'm using out-of-bound means to notify the content hash.
I've noticed that after a few seconds of fetching the file by go-ipfs the js-ipfs app in Chrome browser starts to have very high CPU usage between 50-100%.

Steps to reproduce the error:

Use webpack example app to publish a file (photo is easiest usually) and use gateway.ipfs.io it get it.

I've record a Chrome Performance session you can use to see the high CPU.

@daviddias daviddias added kind/bug A bug in existing code (including security flaws) exp/expert Having worked on the specific codebase is important help wanted Seeking public contribution on this issue P1 High: Likely tackled by core team if no one steps up labels Jan 25, 2018
@daviddias
Copy link
Member

Hi @ido-ran. Thanks for the report. This is a known "situation", we need to get better and perf and benchmarking to make sure that we don't waste unnecessary resources. Unfortunately, the Browser is a really limited environment and so we need to be super careful.

@daviddias daviddias added status/ready Ready to be worked exp/wizard Extensive knowledge (implications, ramifications) required and removed exp/expert Having worked on the specific codebase is important labels Jan 25, 2018
@daviddias daviddias added the topic/perf Performance label Oct 1, 2019
@SgtPooki SgtPooki moved this to 🥞 Todo in js-ipfs deprecation May 17, 2023
@whizzzkid whizzzkid moved this from 🥞 Todo to 🛑 Blocked in js-ipfs deprecation May 24, 2023
@whizzzkid
Copy link

js-ipfs is being deprecated in favor of Helia. You can #4336 and read the migration guide.

Please feel to reopen with any comments before 2023-06-05. We will do a final pass on reopened issues afterward (see #4336).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
exp/wizard Extensive knowledge (implications, ramifications) required help wanted Seeking public contribution on this issue kind/bug A bug in existing code (including security flaws) kind/wontfix-migration-available P1 High: Likely tackled by core team if no one steps up status/ready Ready to be worked topic/perf Performance
Projects
No open projects
Status: Done
Development

No branches or pull requests

3 participants