Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
A
agkyra
Manage
Activity
Members
Labels
Plan
Issues
0
Issue boards
Milestones
Wiki
Code
Merge requests
0
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Releases
Model registry
Operate
Environments
Monitor
Incidents
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
itminedu
agkyra
Commits
b109c272
Commit
b109c272
authored
9 years ago
by
Giorgos Korfiatis
Browse files
Options
Downloads
Patches
Plain Diff
fix deadlock with get_dir_contents
parent
f10c0b1b
No related branches found
Branches containing commit
No related tags found
Tags containing commit
No related merge requests found
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
agkyra/syncer/localfs_client.py
+2
-1
2 additions, 1 deletion
agkyra/syncer/localfs_client.py
with
2 additions
and
1 deletion
agkyra/syncer/localfs_client.py
+
2
−
1
View file @
b109c272
...
...
@@ -695,6 +695,7 @@ class LocalfsFileClient(FileClient):
return
LocalfsTargetHandle
(
self
,
target_state
)
def
get_dir_contents
(
self
,
objname
):
logger
.
debug
(
"
Getting contents for object
'
%s
'"
%
objname
)
with
TransactedConnection
(
self
.
syncer_dbtuple
)
as
db
:
return
db
.
get_dir_contents
(
self
.
SIGNATURE
,
objname
)
...
...
@@ -708,10 +709,10 @@ class LocalfsFileClient(FileClient):
return
rel_path
=
os
.
path
.
relpath
(
path
,
start
=
self
.
ROOTPATH
)
objname
=
utils
.
to_standard_sep
(
rel_path
)
leaves
=
self
.
get_dir_contents
(
objname
)
if
rec
else
None
with
self
.
probe_candidates
.
lock
()
as
d
:
d
[
objname
]
=
self
.
none_info
()
if
rec
:
leaves
=
self
.
get_dir_contents
(
objname
)
for
leaf
in
leaves
:
d
[
leaf
]
=
self
.
none_info
()
...
...
This diff is collapsed.
Click to expand it.
Preview
0%
Loading
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment