-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Eval Refactor for the better compatibility with all protocols and multithreading #766
Merged
Merged
Changes from 37 commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
033d258
removed unwanted goroutine
c6eb707
more refactor of eval
AshwinKul28 9e498b6
Update cmd_meta.go
AshwinKul28 ea94736
Merge branch 'master' into refactor
AshwinKul28 df58e9a
added lint check
AshwinKul28 030c600
Merge branch 'refactor' of https://github.com/AshwinKul28/dice into r…
AshwinKul28 65d05c0
pkg updates
AshwinKul28 037572c
Update migrated_errors.go
AshwinKul28 2dfd647
Update results.go
AshwinKul28 def50f4
Update results.go
AshwinKul28 32c6682
more fixes
cd5a84a
Update server.go
c76ccca
more fixes :(
AshwinKul28 52bbc97
fixed get
AshwinKul28 9e38ea6
Merge branch 'DiceDB:master' into master
AshwinKul28 9d15f1f
Merge branch 'master' into refactor
AshwinKul28 5474698
fix hoja bhai
AshwinKul28 8a443a3
Update store_eval.go
AshwinKul28 9451016
feat: fix http for eval refactor
pratikpandey21 e4093d5
Merge branch 'refactor' of https://github.com/AshwinKul28/dice into r…
AshwinKul28 afb0d25
added enum
AshwinKul28 a4bf6ca
Merge branch 'DiceDB:master' into master
AshwinKul28 b216f3f
Merge branch 'master' into refactor
ce2a96a
more refactor for http and websocket
AshwinKul28 c2c54f7
Update bit_ops_string_int_test.go
AshwinKul28 213acc7
test cases passed
AshwinKul28 9577e1c
Fixing http QWATCH integration tests
lucifercr07 5697fbb
Fixing Websocket tests
lucifercr07 bd084c2
lint fixes
AshwinKul28 f401596
Update setup.go
AshwinKul28 712bbde
fixed unit tests
AshwinKul28 8ebf0d8
Merge branch 'DiceDB:master' into master
AshwinKul28 e9e9b93
Merge branch 'master' into refactor
AshwinKul28 c1a64d8
unit tests
AshwinKul28 17638dd
fix unit tests
AshwinKul28 70481f3
Update netconn.go
AshwinKul28 95f1d6c
Update max_conn_test.go
AshwinKul28 36009ce
more refactor - clean
AshwinKul28 abd303b
Update errors.go
AshwinKul28 f274a00
Update results.go
AshwinKul28 0921cca
Add integration tests for RESP multi-threaded server
soumya-codes ee4ec24
Merge branch 'DiceDB:master' into master
AshwinKul28 8ba1081
Merge branch 'master' into refactor
AshwinKul28 67b2b92
Update server.go
AshwinKul28 8ecd966
Update epoll_linux.go
AshwinKul28 fc986b5
Update server.go
AshwinKul28 0c425ac
more refactoring!!!!
AshwinKul28 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 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 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 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 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 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 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 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
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.
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.
We shouldn't check max connections limit with the value as 50. On my PC the value was 16514. Removing this case for now, we will add it later.