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.

Sign up for GitHub

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

Jump to bottom

Improve configuration for more concurrent requests #338

Merged
merged 6 commits into from
Dec 8, 2016
Merged

Improve configuration for more concurrent requests #338

merged 6 commits into from
Dec 8, 2016

Conversation

benjamingeer
Copy link

@benjamingeer benjamingeer commented Dec 2, 2016 •
edited
Loading

This configures Akka HTTP to handle a large number of concurrent connections, and adds a script to configure the Mac OS X kernel to allow more concurrent connections during testing.

@benjamingeer benjamingeer added the enhancement improve existing code or new feature label Dec 2, 2016
@benjamingeer benjamingeer added this to the Beta Release milestone Dec 2, 2016
@benjamingeer benjamingeer mentioned this pull request Dec 2, 2016
Closed
Merge branch 'develop' into wip/akka-http-concurrency
050e170 
# Conflicts:
#	webapi/src/main/resources/application.conf
@tobiasschweizer
Copy link
Contributor

man sysctl on mac os sierra says:

The -w option has been deprecated and is silently ignored.

@tobiasschweizer
Copy link
Contributor

but the output seems to be OK:

kern.ipc.somaxconn: 128 -> 12000
net.inet.tcp.msl: 15000 -> 1000
net.inet.ip.portrange.first: 49152 -> 32768

@tobiasschweizer
Copy link
Contributor

tobiasschweizer commented Dec 8, 2016 •
edited
Loading

The problem still occurs:

[2016-12-08 17-23-35] [SIPI:ERROR] Server.http() failed: Error #1391: Couldn't read HTTP header: Connection dropped by peer!
[2016-12-08 17-23-35] [SIPI:ERROR] GET: /knora/5nQDbtsoYV2-EoCxKE3glXm.jpg/full/full/0/default.jpg failed: Unknown error! Unauthorized access!

After running the script the thumbnails were not served at all sometimes.

I experienced this which is new to me:

[2016-12-08 17-26-25] [SIPI:ERROR] ERROR IN TIFF! Module: ./images/knora/35A8WxFj3qT-ExxBEo12aEr.jpg Error: Not a TIFF or MDI file, bad magic number 55551 (0xd8ff)
[2016-12-08 17-26-25] [SIPI:ERROR] ERROR IN TIFF! Module: ./images/knora/G6H5H4ffEvf-Cy7I2PsyJ2b.jpg Error: Not a TIFF or MDI file, bad magic number 55551 (0xd8ff)
[2016-12-08 17-26-25] [SIPI:ERROR] GET: /knora/35A8WxFj3qT-ExxBEo12aEr.jpg/full/full/0/default.jpg failed: Internal Server Error! SIPI-IMAGE-ERROR at [/Users/tobi/gitreps/Sipi_github/src/SipiImage.cpp: #348]: Could not read file "./images/knora/35A8WxFj3qT-ExxBEo12aEr.jpg"!
[2016-12-08 17-26-25] [SIPI:ERROR] GET: /knora/G6H5H4ffEvf-Cy7I2PsyJ2b.jpg/full/full/0/default.jpg failed: Internal Server Error! SIPI-IMAGE-ERROR at [/Users/tobi/gitreps/Sipi_github/src/SipiImage.cpp: #348]: Could not read file "./images/knora/G6H5H4ffEvf-Cy7I2PsyJ2b.jpg"!

@benjamingeer
Copy link
Author

In that case I have no idea what's causing #298. Perhaps it's a Sipi bug after all.

@tobiasschweizer
Copy link
Contributor

Ok, I think you can merge your changes to develop and we have to dig into the issue together with Lukas

tobiasschweizer
tobiasschweizer reviewed Dec 8, 2016
View reviewed changes
Copy link
Contributor

@tobiasschweizer tobiasschweizer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please merge

@benjamingeer benjamingeer merged commit 6a3b615 into develop Dec 8, 2016
@benjamingeer benjamingeer deleted the wip/akka-http-concurrency branch December 8, 2016 16:59
SepidehAlassi added a commit that referenced this pull request Dec 13, 2016
@SepidehAlassi
Merge branch 'develop' into wip/beol_biblio
ae34847 
* develop:
  Improve configuration for more concurrent requests (#338)
  Upgrade to Scala 2.12 (#343)
  Fix various inconsistencies in knora-base.ttl and docs. (#330)
  Fix language switching and login/logout in SALSAH (#331)
  Graph data API operation (#267)
  Allow connections over HTTPS (#332)
  Update README.md
  Add link to GraphDB.
  fix (salsah): Add missing JavaScript “var”.
  fix (webapi): Fix RouteUtilV1 so it returns HTML instead of JSON when requested (#326)
  Ontology and documentation fixes. (#325)
  fix (salsah): Create multiple links in a resource, and other bug fixes (#315)
  feature (webapi): Make TransformData fix “test” labels on regions. (#311)
  Fixes for subclasses and subproperties (#303)
  Add script to update docs on knora.org (#300)
  docs: Make trivial change for testing purposes.
  fix (webapi): CORS problems (#307)
  Use inference to optimise searches on GraphDB (#301)
@benjamingeer benjamingeer mentioned this pull request Jul 5, 2019
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tobiasschweizer tobiasschweizer tobiasschweizer left review comments

Assignees

@benjamingeer benjamingeer

Labels
enhancement improve existing code or new feature
Projects
None yet
Milestone
Beta Release
Development

Successfully merging this pull request may close these issues.
3 participants
@benjamingeer @tobiasschweizer @kilchenmann