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

MsSql/PgSql : Add Connection Pooling for optimization #31

Closed
Aniruddh25 opened this issue Sep 22, 2021 · 2 comments
Closed

MsSql/PgSql : Add Connection Pooling for optimization #31

Aniruddh25 opened this issue Sep 22, 2021 · 2 comments
Labels
enhancement New feature or request mssql pgsql mysql an issue that applies to all relational databases, same as labeling with `mssql` `mysql` and `pgsql` perf

Comments

@Aniruddh25
Copy link
Contributor

No description provided.

@Aniruddh25 Aniruddh25 assigned Aniruddh25 and unassigned Aniruddh25 Sep 27, 2021
@jarupatj jarupatj added this to the M0 milestone Nov 2, 2021
@Aniruddh25 Aniruddh25 added the mssql pgsql mysql an issue that applies to all relational databases, same as labeling with `mssql` `mysql` and `pgsql` label Nov 9, 2021
@JelteF
Copy link
Contributor

JelteF commented Dec 28, 2021

I don't think this should be M0. The tooling team can start without having this. So changing this to M1. @jarupatj, please comment if you disagree.

@JelteF JelteF modified the milestones: M0, M1 Dec 28, 2021
@Aniruddh25 Aniruddh25 added the enhancement New feature or request label Feb 14, 2022
@Aniruddh25 Aniruddh25 modified the milestones: M1, M2 Apr 16, 2022
abhishekkumams pushed a commit that referenced this issue Jul 25, 2022
abhishekkumams added a commit that referenced this issue Aug 3, 2022
* Initial commit

* added first design proposal

* CODE_OF_CONDUCT.md committed

* LICENSE committed

* README.md updated to template

* SECURITY.md committed

* SUPPORT.md committed

* new console app

* initial config generator

* updated implementation proposal

* Update README.md

* adding .gitignore

* updating gitignore

* removing compiled files

* added support for command-add

* updating gitignore

* cleaning files and adding todos

* Update README.md

* updated file path for config generation

* using classes of hawaii-gql

* adding design_doc and added update command

* removing temp file

* updating readme

* updating readme

* removing temp files

* removing temp files

* updating README

* removing temp files

* fixing the update command

* removing temp files

* Added relationship compatibility

* updated ReadMe

* removing temp files

* updating gitignore

* update README.md (#5)

* Update README.md (#8)

* Update README.md

* Update README.md

* Fix for connection-string issue (#9)

* fix connection-string issue

* removing temp files

* removing resolver config as it is specific to cosmosDB

* updating .gitignore

* updating .gitignore

* Dev/abhishekkuma/code refactoring (#16)

* fix connection-string issue

* removing temp files

* Use Action class from Runtime Config

* removing temp files

* minor fix

* code refactoring

* added summary for all the methods

* making mappings.field optional (#13)

* Update default host global settings (#19)

* updated default host global settings

* updating default value of Cors.Origin

* fixing spelling errors

* Add support for linking object in relationship (#22)

* adding support for linking object in relationship

* updating validation checks

* fixing grammatical errors

* Add CODEOWNERS file (#24)

* added CODEOWNERS file

* updating names

* adding Davide as code owner

* Update version to 0.0.2 (#25)

* updating version to 0.0.2

* fix indentation

* Add commands in help window (#27)

* added commands in help window

* updated help screen

* updated help screen

* fixing typo

* Update README.md (#26)

* Update README.md

Updating Install Information in Readme.

* Update README.md

Co-authored-by: Aniruddh Munde <anmunde@microsoft.com>

Co-authored-by: Aniruddh Munde <anmunde@microsoft.com>

* Updated Typos (#30)

* Quick fix for typo (#31)

* Add editorconfig to auto format code (#35)

* Add editorconfig

* Fix formatting warning

* Fix more warning

* use is null

* Update CommandLineOptions property to start with capital letter.

* Add skeleton tests project (#36)

* Add tests project

* Update gitignore

* fixing update command for permissions (#34)

* Refactoring commandline handling options. (#37)

**What are the changes**
This change refactors the code to use Verb attribute https://github.com/commandlineparser/commandline/wiki/Verbs
to link command line options to command line operation.

The main change are in CommandLineOptions.cs. With this change, we can use the default help print from Commandline library. We do not need to write our own help text.
Remove Operations.cs and Commandlinehelp.cs. I fold the functionality to program.cs and configgenerator.cs.

Make options class immutable. https://github.com/commandlineparser/commandline/wiki/Immutable-Options-Type

Make database type and host option explicit type from Enum instead of string.

**Test**
Manual testing. I don't change how the config is generated. Same variables are passed to config generator.

* Update cosmosDB options (#29)

* updated cosmosDB options

* fix formatting

* making code more modular

* Update src/Models/CommandLineOptions.cs

Co-authored-by: Aniruddh Munde <anmunde@microsoft.com>

* updating test name

* refactoring code

* refactoring code

Co-authored-by: Aniruddh Munde <anmunde@microsoft.com>

* Move relationship related options to update. (#46)

* Update README.md (#51)

* Refactor config generator for Init and Add command (#41)

* Add tests for initConfig and addEntity. Refactor code a bit to make it more readable.

* Add comments.

* More verbose with arg passing.

* Format code.

* Add more tests for AddEntity.

* resolving comments

* removing temp changes

* small fix

Co-authored-by: Abhishek Kumar <abhishekkuma@microsoft.com>

* fixed tests (#55)

* Update README.md (#59)

updating commands from --permission to --permissions

* Simplify Update Entity logic (#52)

* replace name to config

* symplifying update logic

* adding validations

* added more tests

* resolved comments

* adding test description

* updating file names for Add and Update EntityTests

* adding comments and tests

* adding class comments

* adding class comments

* adding more tests and comments

* updated accessebiity of some methods

* updating tests

* adding TODO for policy support

* updating few comments

* updated function summary

* minor change

* fixed comments

* refactoring changes

* using csproj instead of dll files (#62)

* Create CI pipeline (#56)

* Create main.yml

* Update main.yml

* adding Datagateway.Config.csproj

* fixing path

* fixing tests

* add tests validation in pipeline

* adding formatting check

* updating pipeline

* removing temp changes

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* fix

* updating pipeline

* updating pipeline files

* updating pipeline files

* updating pipeline

* updating pipeline

* updating pipeline

* updating pipeline

* fix pipeline

* fix pipeline

* fix pipeline

* fix pipeline

* fix pipeline

* updating runner

* updating runner

* adding pipeline for ubuntu

* fixing build job name

* using matrix for running multiple os

* updating .editorconfig

* publishing code coverage

* updating coverage directory

* getting dir info

* fixing pipeline

* updating test script

* fixing code coverage path

* fixing code coverage path

* fixing code coverage issue

* fixing code coverage issue

* fixing code coverage issue

* adding deploy steps for the nuget package

* adding deploy steps for the nuget package

* add source path for nuget package

* add source path for nuget package

* fix source path for nuget package

* fix test

* updating format check

* updating format check

* updating format check

* updating format check

* updating format check

* updating format check

* updating format check

* updating format check

* testing submodules

* fixing yaml

* fixing yaml

* fixing yaml

* fixing yaml

* updating .gitmodules

* updating .gitmodules

* updating .gitmodules

* checking submodules

* checking submodules

* checking submodules

* checking submodules

* checking submodules

* adding submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* testing submodules

* removing temp files

* updating yaml

* updating yaml

* updating tests

* updating tests

* testing workflow_dispatch

* testing workflow_dispatch

* refactoring changes

* testing coverage report to pr

* testing coverage report to pr

* testing coverage report to pr

* testing coverage report to pr

* testing coverage report to pr

* testing coverage report to pr

* testing coverage report to pr

* fix coverage report to pr

* fix coverage report to pr

* fix coverage report to pr

* testing different test reported

* testing different test reported

* testing coverage summary

* fix pipeline

* updating comments

* fixing coverage report

* removing temp files

* removing submodules coverage

* removing unused import

* fixing pipeline

* cleaning comments

* Add support for custom graphql singular plural types (#65)

* adding support for custom graphql singular plural types

* fixed formatting

* fixing pipeline

* updating comments

* removing redundant steps in pipeline

* Adding test for Case Sensitive Entity Name (#64)

* Adds test for entity creation

* Adds change to create the options differently

* Adding change to summary and comments

* Removing jwt from the config json strings

* Changing the default rest api endpoint

* Builds config differently to avoid duplication

* Fixing formatting issues

* Preventing users from creation Permissions with Invalid Actions (#70)

* adding tests

* simplyfying code

* added a new test

* simplifying code

* Changing true/false to be treated as a Boolean when used with --rest and --graphql options  (#74)

* Making true/false bool in --rest/graphql options

* addressing review comments

* checking in latest hawaii-engine commits

* addressing review comments

* adding a check for serialization of entity.graphql value

* Add support for Policy in Permission Actions (#71)

* added support for policy

* fix formatting

* added tests

* fix build failure

* fix formatting

* adding comments

* removed redundant code

* updating tests

* updating authentication provider

* updating tests

* updating editorconfig

* updating submodules

* updating submodules

* fixing some dependencies

* Add support for mappings (#76)

* adding test

* fix formatting

* fix error

* fix formatting

* updating option description

* updating comment

* fixing build

* Add support for cors origin (#77)

* added support for cors-origin

* fix formatting

* fix test

* fix formatting

* Prevent user from adding relationship in cosmos db (#79)

* preventing user from creating relationship in cosmosdb

* updating test

* Using IEnumerables for Options (#80)

* adding changes for using ienumerables for options

* removes redundant conversion

* Adding checks for empty items

* fixing console output messages

* adding empty checks for ienumerable options in addition to null

* checking in hawaii-engine sub-module

* Checking in changes from engine

* fixes formatting

* adding test for update command with multiple mapping fields

* fix formatting

* removing extra spaces

* incorporating PR review suggestions

* checnking-in latest changes from hawaii engine

* adding changes to remove empty mappings property in config json

* minor correction in the test command

* adding a test to validate the config json generated

* minor correction in the test

* New Release 0.0.4 (#84)

* Fix serialization issue with special character (#87)

* fixed serialization issue with special charachter

* updating test

* fix formatting

* removing temp changes

* fix test

* updating test

* updating test

* updating test summary

* fix pipeline

* Renaming the option from --mapping.fields to --relationship.fields (#86)

* renaming command option from mapping.fields to relationship.fields

* changing sample commadns in README fiel

* adding latest changes from hawaii engine

* adding latest changes from engine

* updating tests to use relationship.fields

* merge cli into engine

* updating csproj file

* removing cli sln file

* removing duplicate files

* fix formatting

* fix formatting

* adding cli tests into build pipeline

* fix format

* updating folder name for cli-tests

* updating csproj filename for hawaii-cli

* updating csproj filename for hawaii-cli

* fixing build

* fix format

* updating pipeline to create nuget package

* publish nuget packe to artifact

* fix build pipeline

* testing

* testing

* testing

* fixing path for exe

* fixing path for exe

* updating feed for nuget package

* commenting nuget push

* using dependency version from Directory.Build.Props

* testing nuget push

* fixing publish of nuget package

* Update CLI to use Operation Enum.

* Fix build break in tests.

* Verify that operation is valid for action.

* Remove unnessary using.

* updating pipeline

* updating nuget command in pipeline

* updating nuget command in pipeline

* updating nuget command in pipeline

* updating nuget command in pipeline

* testing nupkg creation

* testing nupkg creation

* testing nupkg creation

* testing nupkg creation

* testing nupkg creation

* testing nupkg creation

* testing nupkg creation

* testing nupkg creation

* testing nupkg creation

* testing nupkg publishing

* updating pipeline for nuget push

* fixing pipeline

* fixing pipeline

* fixing pipeline

* fixing pipeline

* fixing pipeline

* fixing pipeline

* fixing pipeline

* fixing pipeline

* fix formatting

* fix formatting

Co-authored-by: Microsoft Open Source <microsoftopensource@users.noreply.github.com>
Co-authored-by: Sajeetharan <sajeetharan@outlook.com>
Co-authored-by: Aniruddh Munde <anmunde@microsoft.com>
Co-authored-by: Chris LaFreniere <40371649+chlafreniere@users.noreply.github.com>
Co-authored-by: Jarupat Jisarojito <guitarnana@gmail.com>
Co-authored-by: Shyam Sundar J <shyam.sundar.jaganathan@gmail.com>
Co-authored-by: Shyam Sundar J <shyamsundarj@microsoft.com>
@Aniruddh25 Aniruddh25 modified the milestones: Oct2022, Sept2022 Aug 24, 2022
@Aniruddh25
Copy link
Contributor Author

Duplicate of #658

@Aniruddh25 Aniruddh25 marked this as a duplicate of #658 Aug 24, 2022
@Aniruddh25 Aniruddh25 closed this as not planned Won't fix, can't repro, duplicate, stale Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request mssql pgsql mysql an issue that applies to all relational databases, same as labeling with `mssql` `mysql` and `pgsql` perf
Projects
None yet
Development

No branches or pull requests

4 participants