HUE-8580 [editor] Fix JDBC assist.

Review Request #13365 - Created Sept. 19, 2018 and submitted

Jean-Francois Desjeans Gauthier
hue
master
HUE-8580
hue
jgauthier
commit 41441720ccc2f7cd188154306625921b662f6451
Author: jdesjean <jgauthier@cloudera.com>
Date:   Wed Sep 19 17:00:12 2018 -0700

    HUE-8580 [editor] Fix jdbc assist.

:100644 100644 fc6482206b... 6134402dab... M	apps/beeswax/src/beeswax/templates/execute.mako
:100644 100644 919bfe9588... b84e9c7f46... M	apps/beeswax/src/beeswax/templates/watch_results.mako
:100644 100644 45cc3ff556... 56cc9b61d3... M	desktop/libs/indexer/src/indexer/file_format.py
:100644 100644 f9b713a83f... c56eac254f... M	desktop/libs/notebook/src/notebook/connectors/base.py
:100644 100644 7fdb16b75e... 58daef19f5... M	desktop/libs/notebook/src/notebook/connectors/jdbc.py
:000000 100644 0000000000... 8635fd2818... A	desktop/libs/notebook/src/notebook/connectors/jdbc_teradata.py
:100644 100644 f999414d18... 98d09084c0... M	desktop/libs/notebook/src/notebook/connectors/rdbms.py
:100644 100644 0c017cd205... 1918090cde... M	desktop/libs/notebook/src/notebook/static/notebook/js/notebook.ko.js

Had to rethink the logic, because I realize it wasn't working correctly for all 3 sqoop import types: custom, custom-jdbc, configured

  1. Nice!

    Will be great for querying + infos for sqoop

  2. Interesting that they have 'sample'

  3. Can't just add the 'e' object instead of using sys?

  4. 
      
Review request changed

Status: Closed (submitted)

Loading...