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

Processes #149

Open
wants to merge 37 commits into
base: v22.07.09
Choose a base branch
from
Open

Processes #149

wants to merge 37 commits into from

Conversation

cleahwin
Copy link

@cleahwin cleahwin commented Sep 16, 2022 •
edited by calebwin

Depends on #148

calebwin and others added 30 commits July 14, 2022 09:04
@calebwin
Minor changes for sessions usability
e096125
@calebwin
Implement configure_sampling, get_sampling_config, get_sample_rate, h…
42173a9 
…as_metadata, configure_sampling
@calebwin
Implement get_location_source and get_organization_id
d8ebaf2
@calebwin
Make changes to location constructor and PFs for BDF.jl
fbf6481
@calebwin
Refactor RemoteTableSource into RemoteSource
d5c11b3
@calebwin
Modify BanyanHDF5.jl to use new sample caching system
347d1f8
@calebwin
Modify BanyanImages.jl to use new sample caching system
ac9d1d1
@calebwin
Implement invalidation functions, update exports, add kwargs to all r…
08c5fbf 
…ead functions
@calebwin
Implement parallel SQS data transfer and eliminate AWSSQS.jl dependency
ee901a8
@calebwin
Remove redundant total_memory_usage
8daff0a
@calebwin
Add BanyanONNXRunTime tests for sampling
f6bb46f
@calebwin
Fix some bugs and update Arrow version
1a3832d
@calebwin
Fix send_to_client
47a4823
@calebwin
Fix SamplingConfig serialization
91245c5
@calebwin
Fix bugs
cbcdd7c
@calebwin
Add more options for updating cluster
dfe98d1
@calebwin
Add Arrow import
52be368
@calebwin
Fix bugs
80e2329
@calebwin
Switch to using S3Path where possible
734f8b7
@calebwin
Fix new sampling system for HDF5
498890a
@calebwin
Fix new sampling system for BanyanImages
c627846
@calebwin
Remove print statements
2ae6477
@calebwin
Make start_session create cluster if needed and change default # of w…
28d5894 
…orkers to 150
@calebwin
Add automatic destruction of idle clusters
a995638
@calebwin
Implement all changes to make starting sessions lazy
045f2e0
@calebwin
Add test of latency of session startup and cluster<->client data tran…
9d0af8a 
…sfer
@calebwin
Change API for print_logs and optimize SQS messaging
54db660
@calebwin @cleahwin
Implement all changes to make starting sessions lazy
3cd39d4
@cleahwin
Add create_process in processes.jl
4c22826
@cleahwin
Add get_process, get_processes, delete_process, end_process functions…
2e033fc 
… to processes.jl
@calebwin calebwin force-pushed the cleah/processes branch 2 times, most recently from 80928f3 to 1f0c68a Compare September 16, 2022 17:48
cailinw
cailinw reviewed Sep 16, 2022
View reviewed changes
Banyan/src/utils.jl Outdated Show resolved Hide resolved
cailinw
cailinw reviewed Sep 16, 2022
View reviewed changes
Banyan/src/utils.jl Outdated Show resolved Hide resolved
@calebwin calebwin force-pushed the cleah/processes branch 3 times, most recently from 5b09749 to e8557d1 Compare September 16, 2022 23:25
@calebwin calebwin changed the title [WIP] Processes Processes Sep 17, 2022
cailinw
cailinw reviewed Oct 1, 2022
View reviewed changes
Banyan/src/Banyan.jl
@@ -21,7 +21,7 @@ global NOT_USING_MODULES = String["ProfileView", "SnoopCompileCore"]
using FilePathsBase: joinpath, isempty
using Base: notnothing, env_project_file

using Arrow,
using Arrow
Copy link
Contributor

Choose a reason for hiding this comment

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

Comma needs to remain

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cailinw cailinw cailinw left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@cleahwin @cailinw @calebwin