-
Notifications
You must be signed in to change notification settings - Fork 51
Add Deltalake query support #1023
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
base: main
Are you sure you want to change the base?
Changes from 21 commits
9d2048e
505ddfe
aee0f8c
d5a25b1
2de051d
7d0b8b7
7195adf
33b787f
2664fcd
b498a76
7da6984
948c108
b0aed4f
a35bcb7
9460601
05f1d0e
e685445
bb0b238
dc0c949
fb84d73
5bdacf5
7ee5515
ae7674d
5538c74
f39c0d3
a965255
03b38e7
3a44b4f
b2a832f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -50,6 +50,7 @@ class MAPIClientSettings(BaseSettings): | |
| "condition_mixing_media", | ||
| "condition_heating_atmosphere", | ||
| "operations", | ||
| "batch_id_neq_any", | ||
| "_fields", | ||
| ], | ||
| description="List API query parameters that do not support parallel requests.", | ||
|
|
@@ -87,4 +88,14 @@ class MAPIClientSettings(BaseSettings): | |
| _MAX_LIST_LENGTH, description="Maximum length of query parameter list" | ||
| ) | ||
|
|
||
| LOCAL_DATASET_CACHE: str = Field( | ||
| os.path.expanduser("~") + "/mp_datasets", | ||
| description="Target directory for downloading full datasets", | ||
| ) | ||
|
|
||
| DATASET_FLUSH_THRESHOLD: int = Field( | ||
| 100000, | ||
|
||
| description="Threshold number of rows to accumulate in memory before flushing dataset to disk", | ||
| ) | ||
|
|
||
| model_config = SettingsConfigDict(env_prefix="MPRESTER_") | ||
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
may want to change to just
os.path.expanduser("~/mp_datasets")so thatoscan resolve non-unix-like separators. Or just usepathlib.Path("~/mp_datasets").expanduser()There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good point
7ee5515