Skip to content

Refactoring, multiple timeouts, code hardening and cleanup #19

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
Merged
merged 106 commits into from
Jul 9, 2025
Merged

Refactoring, multiple timeouts, code hardening and cleanup #19

merged 106 commits into from
Jul 9, 2025

Conversation

@a-random-steve
Copy link
Contributor

@a-random-steve a-random-steve commented Jul 8, 2025

No description provided.

stephenatsembit and others added 30 commits January 28, 2025 17:08
@stephenatsembit
set ConfigureAwait(false) for async calls
ebba922
@stephenatsembit
refactor command options, refactor naming, solidify sync/async method…
27e0cc1 
…ology, switch integrations tests to xunit, begin adding db administration methods
@stephenatsembit
add sample appsettings file, remove test file
012d71e
@stephenatsembit
remove bin and obj folders, move defaults to proper end of commandopt…
7ad7210 
…ions tree
@stephenatsembit
add unittest bin/obj, and appsettings to ignore
37475b1
@jeffrey-elliott
runnable state (renaming and test tweaks)
5da9ccc
@jeffrey-elliott
rename, .gitignore, DbId for admin tests
6e14fe9
@a-random-steve
Merge pull request #1 from stephenatsembit/db_admin
01cd335 
Db admin - initial commit
@jeffrey-elliott
added ListDatabases, ListDatabasesNames and tests
4a09303
@stephenatsembit
turn off implicit usings to make .net462 happy
0044e28
@a-random-steve
Merge pull request #2 from stephenatsembit/db_admin
c84088e 
added ListDatabases, ListDatabasesNames and tests
@jeffrey-elliott
added DoesDatabaseExist by string and by guid
23e7e3a
@a-random-steve
Merge pull request #3 from stephenatsembit/db_admin
114a373 
added DoesDatabaseExist by string and by guid
@stephenatsembit
formatting
6c9e310
@stephenatsembit
todo for integration test to check for existence by name
6cb1cb4
@stephenatsembit
utilizing CommandOptions to affect the http calls, adding methods to …
b4233e9 
…check, and list collections, rounding out create collection methods
@jeffrey-elliott @stephenatsembit
added create, drop; consistent naming
ae9082c
@jeffrey-elliott @stephenatsembit
refs and formatting
de696d1
@jeffrey-elliott @stephenatsembit
spacing
76d117a
@stephenatsembit
move test db name into config
3f4b528
@stephenatsembit
handle sync waitfordatabase
4d8614d
@stephenatsembit
cleanup async methods
0ce4e10
@jeffrey-elliott
refs and spacing
d2526fa
@jeffrey-elliott
async delagte for CreateDatabase
ec11ccf
@jeffrey-elliott
added DatabaseCreationOptions
c775326
@jeffrey-elliott
remove t/s code
3024e03
@jeffrey-elliott
added CommandOptions
4bd98d0
@stephenatsembit
finish handling of keyspace for database, naming adjustments
d187fc1
@a-random-steve
Merge pull request #5 from stephenatsembit/database-admin
d102ea2 
Database admin
@stephenatsembit
Merge commit '2334bb53b9027422e19c36f2165408c636ddc279'
42e26c5 
# Conflicts:
#	src/DataStax.AstraDB.DataAPI/Core/DatabaseOptions.cs
#	src/DataStax.AstraDB.DataApi/Admin/AstraDatabasesAdmin.cs
#	src/DataStax.AstraDB.DataApi/Admin/CloudProviderType.cs
#	src/DataStax.AstraDB.DataApi/Collections/Collection.cs
#	src/DataStax.AstraDB.DataApi/Core/CommandOptions.cs
#	src/DataStax.AstraDB.DataApi/Core/CommandUrlBuilder.cs
#	src/DataStax.AstraDB.DataApi/Core/Commands/Command.cs
#	src/DataStax.AstraDB.DataApi/Core/Database.cs
#	src/DataStax.AstraDB.DataApi/Core/DatabaseOptions.cs
#	src/DataStax.AstraDB.DataApi/Core/Extensions.cs
#	src/DataStax.AstraDB.DataApi/DataAPIClient.cs
#	src/DataStax.AstraDB.DataApi/Utils/Extensions.cs
#	src/DataStax.AstraDB.DataApi/Utils/Guard.cs
#	test/DataStax.AstraDB.DataApi.IntegrationTests/AdminFixture.cs
#	test/DataStax.AstraDB.DataApi.IntegrationTests/ClientFixture.cs
#	test/DataStax.AstraDB.DataApi.IntegrationTests/Tests/AdminTests.cs
#	test/DataStax.AstraDB.DataApi.IntegrationTests/Tests/CollectionTests.cs
a-random-steve and others added 28 commits May 22, 2025 13:10
@a-random-steve
Merge branch 'datastax:main' into main
e8161b0
@stephenatsembit
Merge branch 'main' into tables
3710f47 
# Conflicts:
#	src/DataStax.AstraDB.DataApi/Core/Database.cs
#	src/DataStax.AstraDB.DataApi/Core/Query/IQueryRunner.cs
#	src/DataStax.AstraDB.DataApi/Core/Results/ListTableIndexNamesResult.cs
#	src/DataStax.AstraDB.DataApi/Core/Results/ListTableNamesResult.cs
#	src/DataStax.AstraDB.DataApi/Tables/CreateIndexCommandOptions.cs
#	src/DataStax.AstraDB.DataApi/Tables/DropIndexCommandOptions.cs
#	src/DataStax.AstraDB.DataApi/Tables/Row.cs
#	src/DataStax.AstraDB.DataApi/Tables/Table.cs
#	src/DataStax.AstraDB.DataApi/Tables/TableDefinition.cs
#	src/DataStax.AstraDB.DataApi/Tables/TableInsertManyResult.cs
@stephenatsembit
Remove ability to get database by id, add waitForCompletion option to…
03a0860 
… create and drop keyspace commands, misc cleanup and documentation clarification
@stephenatsembit
update admin test
9b92bc0
@stephenatsembit
Finish draft of hybrid search, improvements to find enumerators, upda…
f90ddb9 
…te find to be fully fluent for consistency
@stephenatsembit
remove test run files
b014ff3
@stephenatsembit
add upgrade info to readme
3aa1a41
@stephenatsembit
initial github action
a3011db
@stephenatsembit
update build path
2b8dece
@stephenatsembit
update build path
a2f66ec
@stephenatsembit
fix build path
8154a68
@stephenatsembit
add dry run release ci
3a44bc1
@stephenatsembit
fix yaml syntax
9cc4de1
@stephenatsembit
change tag processing
29953ff
@a-random-steve
Update release.yml
4fa38d3
@stephenatsembit
remove version from csproj as it will be passed in by the ci/cd action
43849e5
@stephenatsembit
cleanup
6596132
@stephenatsembit
updated databaseinfo objects
4737e60
@stephenatsembit
Merge branch 'upstream-main'
78aeaa9 
# Conflicts:
#	src/DataStax.AstraDB.DataApi/Collections/Collection.cs
#	src/DataStax.AstraDB.DataApi/Core/Query/ProjectionBuilder.cs
#	src/DataStax.AstraDB.DataApi/DataAPIClient.cs
@stephenatsembit
merge cleanup
69cfc64
@stephenatsembit
Merge branch 'main' into client-team-feedback
b3f12cc
@stephenatsembit
remove DeleteAll, simplify DatabaseInfo, add missing property to Vect…
b739299 
…orOptions
@stephenatsembit
handle multiple types and levels of timeouts, fixes for untyped tables
5085fb5
@stephenatsembit
code cleanup and refactoring
de0becd
@stephenatsembit
refactor integration tests, fix database creation options
03571e9
@a-random-steve
Merge branch 'datastax:main' into main
d1e0788
@stephenatsembit
Merge commit 'd1e078886a1a0fd84f6e68770e89c416bf663fe9' into client-t…
5c58942 
…eam-feedback
@stephenatsembit
remove dry run for automated nuget build
fe18671
@vkulichenko vkulichenko merged commit 4cf0999 into datastax:main Jul 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@vkulichenko vkulichenko vkulichenko approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@a-random-steve @vkulichenko @stephenatsembit @jeffrey-elliott