-
Notifications
You must be signed in to change notification settings - Fork 150
Storage tables implementation #5137
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
Merged
Merged
Changes from all commits
Commits
Show all changes
227 commits
Select commit
Hold shift + click to select a range
1373a5f
dss
gearama a36fa1f
Merge branch 'main' of https://github.com/gearama/azure-sdk-for-cpp i…
gearama da70d72
first pass at a client and tests
gearama 7958347
constructors work
gearama 912d69b
working request, need to fix the response parsing
gearama 301e0f1
basic tests working for service client
gearama c6a3a65
refactored the constructors
gearama 9c42788
merge main
gearama 326e888
Merge branch 'main' of https://github.com/gearama/azure-sdk-for-cpp i…
gearama 0dd783f
testproxy and tests
gearama b186f7a
format file
gearama 775b95e
got client and one API working
gearama c15971d
list working
gearama 8648540
all apis recorded
gearama 5821d08
new assets.json
gearama 7599d6c
merge from main
gearama 6384947
assets.json regen
gearama 0c33b9f
format json
gearama 861aab0
adding word to cspell
gearama 4d28554
fix build issues
gearama 2e6e8b5
fix one more break
gearama b40fae8
typo
gearama 82ade61
clangs
gearama 7c93ec1
one more issue
gearama 16f817c
more fixezez
gearama 107821c
regen recordings
gearama 0cb6e05
try again
gearama 3551b54
merge main
gearama b9f25bf
clangs
gearama 68bb829
clangs again
gearama e5c9fc3
new assets
gearama 0e27832
new test proxy
gearama 65e3f9e
main merge
gearama a8a6071
regen tests based on new merge
gearama 9fcd0a3
hmmm
gearama 35fc38f
restore
gearama 988d181
add debug env for test proxy
gearama 8e0d3c6
merge main
gearama 7f26b74
resync cmake presets from main
gearama aad5112
gor get and set working
gearama b3c5406
get stats
gearama ea1651a
table client and create
gearama 3911a15
delete
gearama d969432
get/set acls
gearama 94a3468
list tables
gearama 5c4d50f
some cleanup
gearama dfef6e0
Create /update/merge/delete
gearama fcd461f
upsert
gearama 416f642
query entities
gearama 2c01bb2
merge from main
gearama 0eccc33
merge main
gearama 74b7db9
generated tests
gearama f215580
clangs
gearama a0cdc31
some fixes
gearama 5edd4cd
some more errors
gearama 0fec4a5
cspells
gearama 5ad4434
got transaction API working
gearama b314f02
transactions working , need to parse response
gearama 6ab9558
get data out of the response
gearama 3806958
Merge branch 'main' into testBuild2
gearama 9456b2b
clangs
gearama 9ec1ecf
batch1
gearama c134849
regen tests
gearama 063090e
part 2
gearama 4e7fcec
batch3
gearama a4c0eee
batch4
gearama 1f33523
clangs
gearama 0292dc9
regen tests
gearama 4e140c6
regen tests
gearama e774460
try again
gearama e5ecf17
live only
gearama 56231b1
added some more tests for the transaction body
gearama c812039
small refactor for tests
gearama 44653ca
oops
gearama e74e15a
some oter update
gearama 5cca489
darn includes
gearama 7a2d0cb
sddfsd
gearama 6ca1899
erorr fix
gearama 5107f9a
key clients
gearama 14669be
sas
gearama 45bf172
small cleanup
gearama 464a4a9
All tests passing
gearama a488cd8
test recordings
gearama 6c09e6e
Merge branch 'main' into testBuild2
gearama b26749d
clangs
gearama 3c6d8f2
missing ENV for test
gearama ec82c3e
attempt1
gearama 393b85e
retry
gearama 05e563b
couple recordings
gearama 3a3bd66
regen some tests
gearama a228526
clangs
gearama 998b47c
again
gearama 54647b4
again
gearama 2c6d13d
comments
gearama f43ebbe
comments
gearama 97c5d6d
headers and footers
gearama c21ae25
some cleanjup
gearama cf42a15
Move folders step1
gearama 494a587
revert assets and test resources in storage
gearama b137c82
revert to main
gearama 592506f
cspell
gearama 992423d
readme
gearama f55695e
comments
gearama c29d6e8
Merge branch 'main' of https://github.com/gearama/azure-sdk-for-cpp i…
gearama 46819a8
revert ci.yml
gearama 9426cd9
liveonly test
gearama ec055dc
clangs
gearama 14436d0
camke
gearama 779f8bb
ci2
gearama ec38239
try again
gearama c57a5df
fdssfs
gearama 4cd3987
fdsfsd
gearama c4127ae
some cleanup
gearama 1228ec8
dasda
gearama 4112448
dsdsds
gearama d97852f
asda
gearama 93ac78e
assets
gearama 1e2c26d
comments
gearama 58a5f26
a few more bits and pieces
gearama afa4b7d
some other updates for cspell
gearama 5518783
typo
gearama 7be0a5d
more docs
gearama 21c6072
spell
gearama bffbaea
another doc error
gearama 7cce587
clangs docs
gearama 02e4672
docs
gearama f0db83f
test cov
gearama 78aa67f
dasdas
gearama b624ab7
hg
gearama 8b4a414
stress test
gearama 0a52782
cspell
gearama 3a7b25c
Update sdk/storage/azure-storage-common/test/ut/CMakeLists.txt
gearama f20ca0b
Update sdk/storage/azure-storage-common/test/ut/shared_key_policy_lit…
gearama 01b63cf
Update sdk/tables/azure-data-tables/CHANGELOG.md
gearama 8306770
Update sdk/tables/azure-data-tables/CMakeLists.txt
gearama 8557465
Update sdk/tables/azure-data-tables/test/stress/scenarios-matrix.yaml
gearama e817977
Update sdk/tables/azure-data-tables/test/stress/Dockerfile
gearama 028cd64
PR comments
gearama c88eebc
respurces cleanup
gearama 7351da0
cleanup 2
gearama cf24df2
respurce fix
gearama 448a32e
revddrt
gearama ec3c87f
weqq
gearama d2dc4d6
oops
gearama f3034ba
try again
gearama 04a8d3c
Update sdk/tables/azure-data-tables/vcpkg/vcpkg.json
gearama e47025a
try
gearama c4efed7
test fix
gearama 2b492f5
samples
gearama 4567d98
readmes
gearama c78119b
readmes
gearama d70c114
readmes
gearama 07577b9
api view settings
gearama 28cd39b
qualify friends
gearama 270a109
try
gearama d0e4b86
moving usings
gearama 0fae379
refactor for API View tool
gearama 2b04a0d
some more
gearama e6e7e5f
Merge branch 'main' into testBuild2
gearama 303cf68
clangs
gearama 5e16cb2
Update sdk/tables/README.md
gearama e3143bb
Update sdk/core/azure-core/inc/azure/core/http/http.hpp
gearama 1890ed0
Update sdk/core/azure-core/src/http/transport_policy.cpp
gearama b19b7f7
Update sdk/tables/azure-data-tables/inc/azure/data/tables/rest_client…
gearama e7a7084
Update sdk/tables/azure-data-tables/CHANGELOG.md
gearama 3badb0f
Update sdk/tables/azure-data-tables/test/ut/CMakeLists.txt
gearama 40854af
Update sdk/tables/azure-data-tables/src/rest_client.cpp
gearama e9313e4
some comments
gearama fd9ba18
commit thingy
gearama 1eeab09
comments
gearama 5d4dce8
comments part N
gearama 423c775
clang
gearama 6cd43da
try now
gearama 1546939
clang
gearama 53302cf
comments
gearama 6c119aa
update file names
gearama c0e42a0
update test to check for the final signature
gearama 2f6106f
ccc
gearama 6a890db
rename shared key policy lite
gearama af625a9
cleanup some includes
gearama d05be3a
add a couple more tests
gearama 08f92c4
small update
gearama 25b33fa
clang
gearama 741e889
increase test limits
gearama a3ef35e
xml transition
gearama d7fc970
service_version_policy
gearama 475b507
shared creds and key policy lite
gearama bb56537
switch to secondary policy
gearama 5e9d40c
code done
gearama 6169929
remove last storage vestiges
gearama 5c30f06
put back some code
gearama 36c8b4f
readme/changelog
gearama 751419e
Update sdk/tables/azure-data-tables/src/serializers.cpp
gearama 13676a7
Add the shared key policy for Larry
gearama 4573e2f
Larry comments
gearama 3df4422
curlybracify the initializers
gearama 4b0ccda
Update sdk/core/azure-core-xml/inc/azure/core/xml/dll_import_export.hpp
gearama 86adb13
Update sdk/core/azure-core-xml/vcpkg.json
gearama 4362e9a
Update sdk/tables/azure-data-tables/inc/azure/data/tables.hpp
gearama ce6e070
Rick's comments one more namespace change for serializers
gearama cf7c7ec
Merge pull request #1 from gearama/removeStorageDep
gearama 843bea5
whiteline
gearama 2993391
typo
gearama 543b232
oops
gearama db86043
includes and clangs
gearama 01ee538
cleanup
gearama 762d6be
jhfjdhjfd
gearama bcfd3f0
hjkjdhf
gearama 740d6c2
daas
gearama ac9a7d0
Anton's comments
gearama 106307c
clang
gearama 7184682
revert some changes
gearama 74c07fd
fdss
gearama 8cdb465
Merge branch 'Azure:main' into testBuild2
gearama 89a9fbe
Update sdk/core/azure-core-xml/CHANGELOG.md
gearama e448acc
everything works
gearama 7c6bc45
Merge branch 'testBuild2' of https://github.com/gearama/azure-sdk-for…
gearama 92aea38
clangs and vcpkg test
gearama bd2b299
test remove azure-core-xml
gearama 3f0fb48
vcpkg json
gearama ebb82b4
remove the code
gearama 6bd5eef
Merge branch 'main' of https://github.com/gearama/azure-sdk-for-cpp i…
gearama 5cde47c
merge main
gearama f3f56f2
saa
gearama c5272ba
wqw
gearama 9d7f5f2
last comments
gearama 998070d
clang
gearama File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -359,4 +359,4 @@ build/ | |
| # Default Assets restore directory | ||
| .assets | ||
|
|
||
| CMakeUserPresets.json | ||
| CMakeUserPresets.json | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| # Copyright (c) Microsoft Corporation. | ||
| # Licensed under the MIT License. | ||
|
|
||
| cmake_minimum_required (VERSION 3.13) | ||
|
|
||
| project (azure-data LANGUAGES CXX) | ||
gearama marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| add_subdirectory(azure-data-tables) | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| { | ||
| "AssetsRepo": "Azure/azure-sdk-assets", | ||
| "AssetsRepoPrefixPath": "cpp", | ||
| "TagPrefix": "cpp/tables", | ||
| "Tag": "cpp/tables_a490279ed7" | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,7 @@ | ||
| # Release History | ||
|
|
||
| ## 1.0.0-beta.1 (Unreleased) | ||
|
|
||
| ### Features Added | ||
|
|
||
| - Initial release |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,131 @@ | ||
| # Copyright (c) Microsoft Corporation. | ||
| # Licensed under the MIT License. | ||
|
|
||
| list(APPEND CMAKE_MODULE_PATH "${CMAKE_CURRENT_SOURCE_DIR}/../../../cmake-modules") | ||
| include(AzureVcpkg) | ||
| az_vcpkg_integrate() | ||
|
|
||
| cmake_minimum_required (VERSION 3.13) | ||
| project(azure-data-tables LANGUAGES CXX) | ||
gearama marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| set(CMAKE_CXX_STANDARD 14) | ||
| set(CMAKE_CXX_STANDARD_REQUIRED True) | ||
| set(CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS ON) | ||
|
|
||
| option(FETCH_SOURCE_DEPS "build source dependencies" OFF) | ||
|
|
||
| include(AzureVersion) | ||
| include(AzureCodeCoverage) | ||
| include(AzureTransportAdapters) | ||
| include(AzureDoxygen) | ||
| include(AzureGlobalCompileOptions) | ||
| include(AzureConfigRTTI) | ||
| include(AzureBuildTargetForCI) | ||
| include(CreateMapFile) | ||
|
|
||
| if(FETCH_SOURCE_DEPS) | ||
| set(AZ_ALL_LIBRARIES ON) | ||
| include(FolderList) | ||
| SetCompileOptions(DATA_TABLES) | ||
| GetFolderList(DATA_TABLES) | ||
| foreach(oneFolder IN LISTS BUILD_FOLDERS) | ||
| message("add folder ${oneFolder}") | ||
| add_subdirectory(${oneFolder} EXCLUDE_FROM_ALL) | ||
| endforeach() | ||
| elseif(NOT AZ_ALL_LIBRARIES) | ||
| find_package(azure-core-cpp CONFIG QUIET) | ||
| if(NOT azure-core-cpp_FOUND) | ||
| find_package(azure-core-cpp REQUIRED) | ||
| endif() | ||
| endif() | ||
|
|
||
| set( | ||
| AZURE_DATA_TABLES_HEADER | ||
| inc/azure/data/tables/account_sas_builder.hpp | ||
| inc/azure/data/tables/dll_import_export.hpp | ||
| inc/azure/data/tables/internal/cryptography/hmacsha256.hpp | ||
| inc/azure/data/tables/internal/cryptography/url_encode.hpp | ||
| inc/azure/data/tables/internal/policies/service_version_policy.hpp | ||
| inc/azure/data/tables/internal/policies/shared_key_lite_policy.hpp | ||
| inc/azure/data/tables/internal/policies/shared_key_policy.hpp | ||
| inc/azure/data/tables/internal/policies/switch_to_secondary_policy.hpp | ||
| inc/azure/data/tables/internal/policies/tenant_bearer_token_policy.hpp | ||
| inc/azure/data/tables/internal/policies/timeout_policy.hpp | ||
| inc/azure/data/tables/internal/serializers.hpp | ||
| inc/azure/data/tables/internal/xml_wrapper.hpp | ||
| inc/azure/data/tables/models.hpp | ||
| inc/azure/data/tables/rtti.hpp | ||
| inc/azure/data/tables.hpp | ||
| inc/azure/data/tables/tables_clients.hpp | ||
| inc/azure/data/tables/transactions.hpp | ||
| inc/azure/data/tables/credentials/shared_key_credential.hpp | ||
| ) | ||
|
|
||
| set( | ||
| AZURE_DATA_TABLES_SOURCE | ||
| src/account_sas_builder.cpp | ||
| src/credentials/shared_key_credential.cpp | ||
| src/cryptography/hmacsha256.cpp | ||
| src/policies/shared_key_lite_policy.cpp | ||
| src/policies/shared_key_policy.cpp | ||
| src/policies/switch_to_secondary_policy.cpp | ||
| src/policies/tenant_bearer_token_policy.cpp | ||
| src/policies/timeout_policy.cpp | ||
| src/private/package_version.hpp | ||
| src/serializers.cpp | ||
| src/tables_clients.cpp | ||
| src/transactions.cpp | ||
| src/xml_wrapper.cpp | ||
| ) | ||
|
|
||
| add_library(azure-data-tables ${AZURE_DATA_TABLES_HEADER} ${AZURE_DATA_TABLES_SOURCE}) | ||
| create_per_service_target_build(tables azure-data-tables) | ||
| # make sure that users can consume the project as a library. | ||
| add_library(Azure::azure-data-tables ALIAS azure-data-tables) | ||
|
|
||
| if(WIN32) | ||
| target_link_libraries(azure-data-tables PRIVATE bcrypt webservices crypt32) | ||
| else() | ||
| # Required for Hashing (sha ). | ||
| find_package(OpenSSL REQUIRED) | ||
| target_link_libraries(azure-data-tables PRIVATE OpenSSL::SSL) | ||
| find_package(LibXml2 REQUIRED) | ||
| target_include_directories(azure-data-tables SYSTEM PRIVATE ${LIBXML2_INCLUDE_DIRS}) | ||
| target_link_libraries(azure-data-tables PRIVATE ${LIBXML2_LIBRARIES}) | ||
| endif() | ||
|
|
||
| target_include_directories( | ||
| azure-data-tables | ||
| PUBLIC | ||
| $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/inc> | ||
| $<INSTALL_INTERFACE:include> | ||
| ) | ||
|
|
||
| target_link_libraries(azure-data-tables PUBLIC Azure::azure-core) | ||
|
|
||
| get_az_version("${CMAKE_CURRENT_SOURCE_DIR}/src/private/package_version.hpp") | ||
| generate_documentation(azure-data-tables ${AZ_LIBRARY_VERSION}) | ||
|
|
||
| az_vcpkg_export( | ||
| azure-data-tables | ||
| DATA_TABLES | ||
| azure/data/tables/dll_import_export.hpp | ||
| ) | ||
|
|
||
| az_rtti_setup( | ||
| azure-data-tables | ||
| DATA_TABLES | ||
| azure/data/tables/rtti.hpp | ||
| ) | ||
|
|
||
| # coverage. Has no effect if BUILD_CODE_COVERAGE is OFF | ||
| create_code_coverage(tables azure-data-tables azure-data-tables-test "tests?/*;samples?/*") | ||
|
|
||
| if(BUILD_TESTING) | ||
| add_subdirectory(test/ut) | ||
| add_subdirectory(test/stress) | ||
| endif() | ||
|
|
||
| if(BUILD_SAMPLES) | ||
| add_subdirectory(samples) | ||
| endif() | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.