HUE-9549 [SQLAlchemy] API - Support explain

Review Request #15532 — Created Nov. 3, 2020 and updated Nov. 9, 2020, 2:46 a.m.

Information
Owner:	ayush.goyal
Repository:	hue
Branch:	master
Bugs:	HUE-9549
Depends On:
Reviewers
Groups:	hue
People:	Amlesh1902, johan, ranade, romain, Sreenath, yingc

Description

commit 9ca94ac9d65169383ebb7dc7fad974858a904f76
Author: ayush.goyal <ayush.goyal@cloudera.com>
Date:   Tue Nov 3 19:19:51 2020 +0530

    HUE-9549 [SQLAlchemy] API - Support explain

:100644 100644 59a680e976 7308f5c55a M	desktop/libs/notebook/src/notebook/connectors/sql_alchemy.py

Testing Done

Issues

2
0
0
0
All Issues: 2

Description	From	Last Updated
Any chance to make the result processing a one liner/simpler, e.g. similar to in fetch_result() or https://github.com/cloudera/hue/blob/master/apps/beeswax/src/beeswax/server/hive_server2_lib.py#L1397 ?	romain	Nov. 5, 2020, 1:24 p.m.
And move the execute() + result processing here	romain	Nov. 5, 2020, 1:24 p.m.

Ship it! Fix it!

And how about adding a little new test?

https://github.com/cloudera/hue/blob/master/desktop/libs/notebook/src/notebook/connectors/sql_alchemy_tests.py#L222

desktop/libs/notebook/src/notebook/connectors/sql_alchemy.py (Diff revision 1)

Show all issues

Any chance to make the result processing a one liner/simpler, e.g. similar to
in fetch_result()
or
https://github.com/cloudera/hue/blob/master/apps/beeswax/src/beeswax/server/hive_server2_lib.py#L1397

ayush.goyal Nov. 9, 2020, 2:46 a.m.

tried but not know how to do it in one line

desktop/libs/notebook/src/notebook/connectors/sql_alchemy.py (Diff revision 1)
Show all issues
```
And move the execute() + result processing here
```

Diff:

Revision 2 (+22 -1)

Show changes

	desktop/libs/notebook/src/notebook/connectors/sql_alchemy.py
	desktop/libs/notebook/src/notebook/connectors/sql_alchemy_tests.py

You have a pending review.

Review Board 3.0.18

HUE-9549 [SQLAlchemy] API - Support explain

Diff: