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.

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

Simplify Configuration #3035

Merged
merged 7 commits into from
Nov 2, 2022
Merged

Conversation

devinrsmith
Copy link
Member

@devinrsmith devinrsmith commented Oct 26, 2022

Introduces deephaven.cacheDir, deephaven.configDir, deephaven.dataDir, and deephaven.application (and corresponding environment variable alternatives).

During server startup, if not explicitly provided, these values will be chosen based on the current user and operating system conventions.

@devinrsmith devinrsmith added this to the Oct 2022 milestone Oct 26, 2022
@devinrsmith devinrsmith self-assigned this Oct 26, 2022
@devinrsmith devinrsmith force-pushed the workspace-cleanup branch 2 times, most recently from d86677e to 9f8fc7e Compare October 28, 2022 19:02
@devinrsmith devinrsmith changed the title Workspace cleanup Simplify Workspace Oct 28, 2022
Introduces deephaven.cacheDir, deephaven.configDir, and deephaven.dataDir.

During server startup, if not explicitly provided, these values will be choosen based on the current user and operating system conventions.
@devinrsmith devinrsmith changed the title Simplify Workspace Simplify Configuration Oct 28, 2022
@devinrsmith
Copy link
Member Author

follow up #3055

@devinrsmith
Copy link
Member Author

follow up #3054

@devinrsmith devinrsmith merged commit fb385f7 into deephaven:main Nov 2, 2022
@devinrsmith devinrsmith deleted the workspace-cleanup branch November 2, 2022 21:26
@github-actions github-actions bot locked and limited conversation to collaborators Nov 2, 2022
@deephaven-internal
Copy link
Contributor

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants