-
Notifications
You must be signed in to change notification settings - Fork 82
Query Server package code review #286
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
Open
iblislin
wants to merge
66
commits into
djc:master
Choose a base branch
from
iblislin:issue-268
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
66 commits
Select commit
Hold shift + click to select a range
b9ac81b
[server] Introduce query server package
kxepal e38397e
[server] Introduce `BaseQueryServer` and `SimpleQueryServer`
kxepal 3a71bff
[server] mv view.py to server/__main__.py
kxepal 820bda5
[server] Add server test suite
kxepal 55d7088
[server] Test suite for `server.stream`
kxepal dcbbd47
[server] Apply proper API calls to __main__.py
kxepal b47a059
[server] New cmd option `--log-level`
kxepal 76256a1
[server] New cmd option `--allow-get-update`
kxepal ce1a300
[server] New cmd option `--enable-eggs`
kxepal e217c4f
[server] New cmd option `--egg-cache`
kxepal 12893f6
[server] New cmd option `couchdb-version`
kxepal 146d969
[server] `BaseQueryServer.commands` property
kxepal 147364e
[server] Exception handlers for `BaseQueryServer`
kxepal 369f320
[server] Property `BaseQueryServer.state`
kxepal bd98047
[server] Server cmd: `log`
kxepal a0fbacd
[server] Server API: `compile`
kxepal 2f562a7
[server] Resolve dependencies of `compiler`: `maybe_export_egg`
kxepal 3bad349
[server] Resolve dependencies of `compiler`
kxepal 45682c5
[server] Resolve dependencies of `compiler`: `require`
kxepal 906c6b1
[server] Resolve dependencies of `compiler`: `compile_func`
kxepal 37798ba
[server] Server cmd: `add_lib`
kxepal 4349d51
[server] Server cmd: `add_fun`
kxepal 9c30606
[server] Server cmd: `reset`
kxepal c4e152b
[server] Server cmd: `add_ddoc`
kxepal d4f7a01
[server] Server API: `ddoc_cmd`
kxepal 698ee0e
[server] Server cmd: `map_doc`
kxepal d00497d
[server] Server cmd: `reduce`
kxepal bb91838
[server] Server cmd: `rereduce`
kxepal a2c5841
[server] Server cmd: `show_doc`
kxepal 0a457bf
[server] `render_function` required by `show_doc`
kxepal cca018d
[server] `response_with` required by `show_doc`
kxepal dbcc74d
[server] Introduce `MimeProvider`
kxepal b321342
[server] Add `util.OrderedDict` required by `mime`
iblislin 1db4916
[server] Add helper functions in module `mime`
kxepal 91abfb9
[server] Server cmd: `list_old`
kxepal 3f47ac1
[server] Server cmd: `show`
kxepal 9e9422c
[server] Server cmd: `list`
kxepal 2148e00
[server] Server cmd: `update`
kxepal ef9c855
[server] Server cmd: `filter`
kxepal 6cede86
[server] Server cmd: `validate_doc_update`
kxepal 6f365b8
[server] Server cmd: `ddoc shows`
kxepal 8e7b197
[server] Server cmd: `ddoc lists`
kxepal 64c7d14
[server] Server cmd: `ddoc updates`
kxepal 106e528
[server] Server cmd: `ddoc filters`
kxepal e60cf48
[server] Server cmd: `ddoc views`
kxepal 27facf4
[server] Server cmd: `ddoc validate_doc_update`
kxepal 60b8555
[server] Add test suite: 'cli'
kxepal e741a85
[server] unicode anywhere in stream.respond
iblislin cc4798e
[server] Fix setup.py packages options
iblislin 8bd1985
[server] update docstring of __main__ script
iblislin 439af0a
[server] move local import statement to the top
iblislin 39131fd
[server] Fix option string of cli script
iblislin 838c861
[server] Replace try-except block with assertRaises
iblislin 2b84a3a
[server] Apply `None` checking for `qs.log` properly
iblislin c1b50cd
[server] fix typo in compiler.py
iblislin 71db52b
[server] clean up legacy code in compiler.py
iblislin 2b8d345
[server] clean up legacy test case in compiler.py
iblislin 0b99d7b
[server] Reimplement compiler.maybe_b64egg
iblislin b0cc416
[server] Accept empty context in compiler.require
iblislin f0c5054
[server] code clean up in compiler.py
iblislin f373e06
[server] check side effect in test case
iblislin a2ff50d
[server] Remove obsolete var: state.line_length
iblislin f7aee62
[server] Exam the output of user map funcion
iblislin f91a237
[server] check the empty output of map_doc in test case
iblislin 13f6c8d
[server] raise exceptions in try-else block for views.py
iblislin d099408
[server] mime: update DEFAULT_TYPES
iblislin 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
[server] Apply
None
checking for qs.log
properly
- Loading branch information
commit 2b84a3a8122faaa2d5d0cad9528f0ac32d78b6eb
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
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
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.
I guess this will be better.
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.
+1
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.
Thought I would keep test for old style logging.
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.
Ok. I can keep test for old one and add another testing code for new style.
Uh oh!
There was an error while loading. Please reload this page.
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.
Patch V2
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.
+1
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.
Patched @ 2b84a3a