chore: pull changes from origin repo#2
Merged
liranp merged 23 commits intoupwindsecurity:v3from Aug 12, 2025
Merged
Conversation
Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.9.0 to 1.10.0. - [Release notes](https://github.com/stretchr/testify/releases) - [Commits](stretchr/testify@v1.9.0...v1.10.0) --- updated-dependencies: - dependency-name: github.com/stretchr/testify dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…trategies * new option to define the max memory the cache can use in bytes * cache.set updated to cope with memory limitations * options updated to implement the suggested new option * cache impl updated + tests * dependencies updated * imports organized * readme updated to document the new configuration option * better explanation in the example * readme line removed by accident restored * function renamed (typo fixed) * wording fixed * useless sentence removed * costs renamed to cost * cost used in tests simplified * CostCalcFunc renamed to CostFunc * costsCalcFunc renamed to costFunc * updateExpirations moved * totalCost renamed to maxCost * WithTotalCost option renamed to WithMaxCost * example in README updated * signature of the WithMaxCost option changed * readme updated * Item impl updated to hold the current cost and the corresponding cost calculation function; new constructor expecting options * cache impl updated to make use of the new item properties * new item options * tests for the new options * new Item related tests * cache tests fixed to make them compile * item option implementation updated to become a private interface * cache set test updated * made new item related functions private + eviction reason renamed * moved initial item cost calsulation to the constructor function * test functions renamed to reflect the private nature of functions under test * version tracking fixed * small readme update
Bumps [golang.org/x/sync](https://github.com/golang/sync) from 0.9.0 to 0.10.0. - [Commits](golang/sync@v0.9.0...v0.10.0) --- updated-dependencies: - dependency-name: golang.org/x/sync dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Changed applyOptions and the Option[K,V] to be non-allocating by avoiding pointers Added the Test_Get_DoesNotAllocate to make sure that Get does not allocate (it fails before this commit) --------- Co-authored-by: david <dovb@jellydator.dev>
Bumps [golang.org/x/sync](https://github.com/golang/sync) from 0.10.0 to 0.11.0. - [Commits](golang/sync@v0.10.0...v0.11.0) --- updated-dependencies: - dependency-name: golang.org/x/sync dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* GetOrSetFunc. For example, the method would be useful in an expression like this: namedLocks.GetOrSetFunc(name, func()*sync.Mutex{ return new(sync.Mutex) })
* save some memory
* applyOptions accpent struct, not pointer
* Fixed setOpts in SetOrGetFunc. Unit-test for GetOrSetFunc
* simplify our old GetOrSet for re-use logic
Added a new flag to support calling stopped cache cleanup procedures as well as a mutex to control it. Now: - Start method will only run once even with the subsequent calls to it. - Stop will never block, in case the Start was not called or Stop was called before, it will return as the cache cleanup process is not running. Closes #175
Bumps [golang.org/x/sync](https://github.com/golang/sync) from 0.11.0 to 0.14.0. - [Commits](golang/sync@v0.11.0...v0.14.0) --- updated-dependencies: - dependency-name: golang.org/x/sync dependency-version: 0.14.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
emanela
approved these changes
Aug 11, 2025
arnarpall
approved these changes
Aug 12, 2025
|
שׂשששששְׂשׂשְׂששששׂ'''ששש''ש''שׂ'ששששְשששׂששז׳שש2wAp0SL01,א
…On Tue, Aug 12, 2025, 21:28 Liran Polak ***@***.***> wrote:
Merged #2 <#2> into v3.
—
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BFCCW75TBEW6QTXL7AVJZQL3NI55RAVCNFSM6AAAAACDUKOGZ6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJZGEZDANJWHE4DQMA>
.
You are receiving this because your review was requested.Message ID:
***@***.***>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
11 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Mostly irrelevant to us but jellydator#162 looks like it might be a nice free performance improvement.