Skip to content

Commit cf1ab6d

Browse files
authored
fix: get all tags should work for all resources (#175)
Signed-off-by: Tao Feng <[email protected]>
1 parent 369685c commit cf1ab6d

File tree

3 files changed

+6
-5
lines changed

3 files changed

+6
-5
lines changed

metadata_service/proxy/neo4j_proxy.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -673,8 +673,8 @@ def get_tags(self) -> List:
673673
# todo: Currently all the tags are default type, we could open it up if we want to include badge
674674
query = textwrap.dedent("""
675675
MATCH (t:Tag{tag_type: 'default'})
676-
OPTIONAL MATCH (tbl:Table)-[:TAGGED_BY]->(t)
677-
RETURN t as tag_name, count(distinct tbl.key) as tag_count
676+
OPTIONAL MATCH (resource)-[:TAGGED_BY]->(t)
677+
RETURN t as tag_name, count(distinct resource.key) as tag_count
678678
""")
679679

680680
records = self._execute_cypher_query(statement=query,

requirements.txt

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ mock==2.0.0
2828
# Mypy is an optional static type checker for Python.
2929
# License: MIT
3030
# Upstream url: https://github.com/python/mypy
31-
mypy==0.660
31+
mypy==0.782
3232

3333
# Thin-wrapper around the mock package for easier use with py.test.
3434
# License: MIT
@@ -60,7 +60,7 @@ marshmallow-annotations==2.4.0
6060
MarkupSafe==1.1
6161
pytz==2018.4
6262
six==1.11.0
63-
Werkzeug==0.15.3
63+
Werkzeug==0.15.5
6464
wheel==0.33.1
6565
neo4j==1.7.6
6666
neotime==1.7.1
@@ -72,3 +72,4 @@ beaker>=1.10.0
7272
mocket==3.7.3
7373
overrides==2.5
7474
websocket-client==0.56.0
75+
typed-ast==1.4.1

setup.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
from setuptools import setup, find_packages
77

8-
__version__ = '2.5.4'
8+
__version__ = '2.5.5'
99

1010

1111
requirements_path = os.path.join(os.path.dirname(os.path.realpath(__file__)), 'requirements.txt')

0 commit comments

Comments
 (0)