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.

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

render Propfind responses as xml stream #3024

Open
wants to merge 11 commits into
base: edge
Choose a base branch
from
Open

render Propfind responses as xml stream #3024

wants to merge 11 commits into from
+46 −68

Conversation

butonic
Copy link
Contributor

@butonic butonic commented Jul 1, 2022 •
edited
Loading

further performance improvements

  • streaming xml rendering

Not adding much today, but will become more noticable when switching to ListContainerStream

butonic added 7 commits July 6, 2022 19:25
@butonic
update propfind unit tests
130063f 
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic
fix moq propfind comparison in propfind tests
42c94f3 
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic
typos, mimic oc10 not found error message
2d3add9 
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic
add fieldmask to ListContainer and Stat
3ba90e0 
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic
stream xml propfind per resource
90164f9 
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic
indexer: introduce LookupCtx
b887045 
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic
fix build after rebase
9c7fe92 
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic butonic changed the title Skip n stream render Propfind responses as xml stream Jul 6, 2022
butonic added 4 commits July 6, 2022 20:05
@butonic
add changelog
81523f6 
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic
lint
f9c9d08 
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic
Revert "indexer: introduce LookupCtx"
4c453be 
This reverts commit b887045.
@butonic
remove commented code
a5878d2 
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic butonic marked this pull request as ready for review July 11, 2022 06:40
@butonic butonic requested review from a team, labkode, ishank011 and glpatcern as code owners July 11, 2022 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@labkode labkode Awaiting requested review from labkode labkode is a code owner

@ishank011 ishank011 Awaiting requested review from ishank011

@glpatcern glpatcern Awaiting requested review from glpatcern glpatcern is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@butonic