FazBrowse GitHub Viewer | Trending | | Home

Tools: [Download Repo ZIP] [Original HTTPS Page]

SI: Implement put operations by susodapop · Pull Request #67 · databricks/databricks-sql-python · GitHub

SI: Implement put operations - #67

Merged

susodapop merged 41 commits into

sifrom

Dec 30, 2022

Commits on Nov 15, 2022

Basic PUT operation. Currently this never executes because the server
Jesse Whitehouse
committed
28c3a59
Bump Spark CLI service protocol version being used.
Jesse Whitehouse
committed
1b245b1
Log when attempting a staging operation
Jesse Whitehouse
committed
1239def

Commits on Nov 16, 2022

Fix failing unit tests since function signature for ExecuteResponse changed
Jesse Whitehouse
committed
b605cce
Add e2e test for put.
Jesse Whitehouse
committed
3ed84d8
Bail on tests if staging_ingestion_user is not set
Jesse Whitehouse
committed
57b8a34
Black client.py
Jesse Whitehouse
committed
7812278

Commits on Nov 17, 2022

Add unit test that sanity checks _handle_staging_operation is called
Jesse Whitehouse
committed
6b76439
Fix imports so that this module can be run independently:
Jesse Whitehouse
committed
3df7c89
Implement GET operation
Jesse Whitehouse
committed
8f0a02e

Commits on Nov 23, 2022

Refactor client.py into distinct methods for each ingestion command type
Jesse Whitehouse
committed
55525cb
Update pypoetry so I can develop on Python 3.10
Jesse Whitehouse
committed
157ac3d
Applied PR feedback around explicit response codes.
Jesse Whitehouse
committed
0739ccc
Applying PR feedback
Jesse Whitehouse
committed
d3a3651
PR feedback
Jesse Whitehouse
committed
72f917e
Black client.py
Jesse Whitehouse
committed
fba64b7
Refactor e2e test to use a single teste for PUT, GET, and REMOVE
Jesse Whitehouse
committed
c27a3d6
Make REMOVE command work
Jesse Whitehouse
committed
19ca706
These methods don't need to know the `operation`
Jesse Whitehouse
committed
0167bd9
Remove single quote that broke query
Jesse Whitehouse
committed
85e4d7c
Remove unneeded argument
Jesse Whitehouse
committed
713002d
Expect operation to succeed
Jesse Whitehouse
committed
fc06ef8
Black PySQLStagingIngestionTestSuite only
Jesse Whitehouse
committed
cafa17d
Tidy up comments in e2e test
Jesse Whitehouse
committed
a508a1c
Basic e2e test scaffolded in. Currently fails.
Jesse Whitehouse
committed
ce80df0
Only allow ingestion commands when base_uploads_path is specified
Jesse Whitehouse
committed
36885a4
Restrict local file operations to descendents of uploads_base_path
Jesse Whitehouse
committed
c0c09d4

Commits on Dec 20, 2022

Remove per PR feedback
Jesse Whitehouse
committed
f612795
Add check for null local_file per PR feedback
Jesse Whitehouse
committed
e609ef3
Open output stream _after_ successful HTTP request
Jesse Whitehouse
committed
cdbe2d6
Resolve relative paths before comparing row.localFile to uploads_base_path
Jesse Whitehouse
committed
34a0362
Add test that PUT fails if file exists in staging location and OVERWRITE not set
Jesse Whitehouse
committed
c8a64c7
Add tests: operations fail to modify another user's staging location
Jesse Whitehouse
committed
d48d3f3
Add test that ingestion command fails if local file is blank
Jesse Whitehouse
committed
e0037e0
Add test that invalid staging path will fail at server
Jesse Whitehouse
committed
3fa5d84

Commits on Dec 22, 2022

Basic usage example (needs tweaking)
Jesse Whitehouse
committed
4824b68
Add samples of GET and REMOVE
Jesse Whitehouse
committed
469f35f

Commits on Dec 28, 2022

Refactor to allow uploads_base_path to be either a single string object
Jesse Whitehouse
committed
bdb948a

Commits on Dec 29, 2022

Refactor uploads_base_path to staging_allowed_local_path
Jesse Whitehouse
committed
0261b7a

Commits on Dec 30, 2022

Fix mypy static type failures
Jesse Whitehouse
committed
00d8a49
Black src files
Jesse Whitehouse
committed
7a602e6

Back | FazBrowse Home | New Git URL