Skip to content

Commit 1264342

Browse files
authored
Fix detailed mode output for summary requests (#461)
* Fix detailed mode output for %statistics/%summary * update changelog --------- Co-authored-by: Michael Chin <[email protected]>
1 parent b417875 commit 1264342

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

ChangeLog.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
Starting with v1.31.6, this file will contain a record of major features and updates made in each release of graph-notebook.
44

55
## Upcoming
6+
- Fixed detailed mode output for graph summary requests ([Link to PR](https://github.com/aws/graph-notebook/pull/461))
67
- Added more helpful error messaging for `%statistics`/`%summary` ([Link to PR](https://github.com/aws/graph-notebook/pull/460))
78
- Added Neptune Notebook CloudFormation template ([Link to PR](https://github.com/aws/graph-notebook/pull/442))
89

src/graph_notebook/neptune/client.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -759,9 +759,10 @@ def statistics(self, language: str, summary: bool = False, mode: str = '') -> re
759759

760760
if summary:
761761
summary_url = url + '/summary'
762-
if not data['mode']:
763-
data['mode'] = 'basic'
764-
req = self._prepare_request('GET', summary_url, data=json.dumps(data), headers=headers)
762+
if mode:
763+
summary_mode_param = '?mode=' + mode
764+
summary_url += summary_mode_param
765+
req = self._prepare_request('GET', summary_url, headers=headers)
765766
else:
766767
if mode in ['', 'status']:
767768
req = self._prepare_request('GET', url, headers=headers)

0 commit comments

Comments
 (0)