Skip to content

Conversation

@charlie-lobo
Copy link
Contributor

The serialized nature of it meant that even a slightly larger than
expected delay could quickly add up to enough to trigger timeouts
consistently. Since these operations are highly IO bound there's no
reason not to run them all asynchronously.

…arallel.

The serialized nature of it meant that even a slightly larger than
expected delay could quickly add up to enough to trigger timeouts
consistently. Since these operations are highly IO bound there's no
reason not to run them all asynchronously.
@charlie-lobo charlie-lobo force-pushed the carlopez-temporal-parallel-search branch from ef3cb58 to c7e41fd Compare June 24, 2025 21:06
Copy link
Contributor

@Kvadratni Kvadratni left a comment

Choose a reason for hiding this comment

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

seems legit

@charlie-lobo charlie-lobo force-pushed the carlopez-temporal-parallel-search branch from 9dd62d0 to 404e0d4 Compare June 24, 2025 22:06
local directory, and only then will it check for a bundled case.
@charlie-lobo charlie-lobo force-pushed the carlopez-temporal-parallel-search branch from 404e0d4 to 4497e58 Compare June 24, 2025 22:18
@charlie-lobo charlie-lobo merged commit 041ebd5 into main Jun 25, 2025
8 checks passed
@charlie-lobo charlie-lobo deleted the carlopez-temporal-parallel-search branch June 25, 2025 00:22
Kvadratni added a commit that referenced this pull request Jun 25, 2025
ahau-square pushed a commit that referenced this pull request Jun 25, 2025
* origin/main:
  Session file security updates (#3071)
  feat(ui): Add drag and drop support for opening sessions from finder. (#3070)
  Mnovich/fix cli permisisons (#3074)
  copy permission fix for cli (#3073)
  Update download_cli.sh to support arbitrary Goose versions (#3060)
  fix(temporal-service): makes the search for possible temporal paths parallel. (#3062)
  added MCPs menu item (#3066)
  fix missing dependencies for cli (#3065)
  feat: created sub recipe tools (#2982)
  attempt to build CLI native for Windows (#3058)
  docs: typo - wrong extension id (#3063)
  fix(docs): teach Goose how to spell “autonomous” 🪿📚 (#3061)
  feat (cli): list Groq-supported models (#3048)
  Fix session corruption issues (#3052)
  feat: update to get vector db path from env var for vector tool strategy (#3042)
  Fix Linux desktop download links to point to actual desktop versions (#3051)
  fix temporal build for windows (#3045)
  fix cron parsing for windows (#3044)
s-soroosh pushed a commit to s-soroosh/goose that referenced this pull request Jul 18, 2025
…arallel. (block#3062)

The serialized nature of it meant that even a slightly larger than expected delay could quickly add up to enough to trigger timeouts consistently. Since these operations are highly IO bound there's no reason not to run them all asynchronously.

Signed-off-by: Soroosh <soroosh.sarabadani@gmail.com>
cbruyndoncx pushed a commit to cbruyndoncx/goose that referenced this pull request Jul 20, 2025
…arallel. (block#3062)

The serialized nature of it meant that even a slightly larger than expected delay could quickly add up to enough to trigger timeouts consistently. Since these operations are highly IO bound there's no reason not to run them all asynchronously.
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.

3 participants