Skip to content

Create tiered artifact #12

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
Open
wants to merge 87 commits into
base: dummy-tiered-caching-base
Choose a base branch
from
Open

Create tiered artifact #12

wants to merge 87 commits into from

Conversation

@peteralfonsi
Copy link
Owner

@peteralfonsi peteralfonsi commented Dec 4, 2023

Description

Dummy PR to build artifact thru Github Actions for use with CDK

Related Issues

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)
  • Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Peter Alfonsi and others added 30 commits September 28, 2023 16:04
Implemented and tested time took for QuerySearchResult objects for us…
fcd7476 
…e in tiered caching
Moved setting to QueryPhase.execute()
87cbe2a
Moved setting to QueryPhase.execute()
cf18eb9
spotlessCheck
c876de8 
Signed-off-by: Peter Alfonsi <[email protected]>
Added first draft of caching policy setup
93a16d4
@sgup432
Tiered caching framework draft changes
8d6c8c4
initial commit
83590e1
fixed permissions
aa0a5d7
Set up event listener to create RemovalNotification objects, also mis…
41dc035 
…c functions
Added misc getter methods
4ba49af
thread pool stuff
dcb9fad
cleanup
b912527
Removed partial cache files from disk
44d5e61
fixed double counting of on heap hit count
43e2d1d
first attempt at fixing cache closing issue in tests
aa2d28d
misc stuff for testing
354c311
Moved more stuff around for debugging issues
66915b6
fixed various serialization issues relating to silent ehcache put() a…
f985e63 
…nd get() failures
Cleanup
ae300e9
More cleanup
7a7b41f
Merged in Sagar's serialization changes
ec729d0
Part 1 of modifying ehcache to take serializable Key
990ba9c
final attempts at doing it generally before learning that wasnt even …
d21af15 
…needed
Changed Ehcache disk tier implementation to not be generic, fixed per…
d152478 
…sistent cache locking issue
Fixed multiple disk tier instance issue, made spillover test pass
2d200fc
Removed persistence, cleaned up
8f65db4
cleanup
7155afd
added test entries metric for on heap
c760594
Fixed issue where multiple mock nodes on one JVM (in IT tests) would …
8510637 
…cause initialization errors in the cache manager
Merge branch 'framework-serialized' into update_node_stats
0b2e506 
Fixing initialization issue that broke IT tests
Peter Alfonsi added 30 commits November 8, 2023 15:26
Merge commit '157ca6e9557cc5864c45c930be97efa6f17ea269' into cache-value
695521f 
Signed-off-by: Peter Alfonsi <[email protected]>
Almost finished statsholder impl side, found and deleted a lot of old…
8fb86b7 
… tiered stuff in indices package
Finished cachevalue rework + disk stats implementation
900230d 
Signed-off-by: Peter Alfonsi <[email protected]>
Simplified RBM size estimator
07a2e4b 
Signed-off-by: Peter Alfonsi <[email protected]>
Merge branch 'keystore-squashed' into feature/tiered-caching
a9df136
addressed sagar's comments
2f884c2 
Signed-off-by: Peter Alfonsi <[email protected]>
merged in serializer changes
9d88676 
Signed-off-by: Peter Alfonsi <[email protected]>
Fixed broken test
4e8ba0b 
Signed-off-by: Peter Alfonsi <[email protected]>
Merged in cache value changes
41ae53e 
Signed-off-by: Peter Alfonsi <[email protected]>
Re-added and tested serializer for IRC key
09748eb 
Signed-off-by: Peter Alfonsi <[email protected]>
added null check to BR deserializer
8cb83d7 
Signed-off-by: Peter Alfonsi <[email protected]>
Fixed ehcache init issue in test cases
111c4e6 
Signed-off-by: Peter Alfonsi <[email protected]>
cleanup, fixed commented line
55b09fe 
Signed-off-by: Peter Alfonsi <[email protected]>
Fixed issues with closing disk tier in tests, added IT for disk speci…
24a3cb9 
…fic stats

Signed-off-by: Peter Alfonsi <[email protected]>
removed print statement
6a280c6 
Signed-off-by: Peter Alfonsi <[email protected]>
Removed incorrect line from test
175c5c5 
Signed-off-by: Peter Alfonsi <[email protected]>
added required javadoc
af5178f 
Signed-off-by: Peter Alfonsi <[email protected]>
Optimized modulo calculation with bitmask
7d1ce12 
Signed-off-by: Peter Alfonsi <[email protected]>
Addressed Sagar's comments on caching policy/generics
57e948f 
Signed-off-by: Peter Alfonsi <[email protected]>
Addressed Sagar's other comments
c51e0f3 
Signed-off-by: Peter Alfonsi <[email protected]>
removed dead code
100ca7b 
Signed-off-by: Peter Alfonsi <[email protected]>
removed more dead code
c415ec8 
Signed-off-by: Peter Alfonsi <[email protected]>
Changed memory size estimator to reflect optimized modulo
93fa4f8 
Signed-off-by: Peter Alfonsi <[email protected]>
Optimizes modulo using bitmask
433470e
dummy commit
e41321d
Changed github action to run on my fork. REVERT THISgit add .
a51da38
added broken javadoc
f0a26b9
added upload artifact section to workflow, spotlessApply
01bcad7
Added notification for REMOVED case in ehcache event listener
c6a93a4
fixed artifact path
ef019af
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@peteralfonsi @sgup432