HUE-7913 [editor] Provide sample data for variables

Review Request #12519 - Created Feb. 12, 2018 and submitted

Jean-Francois Desjeans Gauthier
commit fc706b82136f6958c5c1f9feaea06a6da3ffc908
Author: jdesjean <>
Date:   Tue Jan 30 11:36:14 2018 -0800

    HUE-7913 [editor] Provide sample data for variables

:100644 100644 1c8de6b519... 0b6049c8aa... M	desktop/core/src/desktop/templates/common_notebook_ko_components.mako
:100644 100644 e4e75893f3... 883a79b134... M	desktop/libs/notebook/src/notebook/static/notebook/css/notebook.css
:100644 100644 a8fcb54524... 55f991395b... M	desktop/libs/notebook/src/notebook/static/notebook/js/notebook.ko.js
:100644 100644 dc5f705980... 656af924c5... M	desktop/libs/notebook/src/notebook/static/notebook/less/notebook.less
:100644 100644 cb07df5378... 8b426c2778... M	desktop/libs/notebook/src/notebook/templates/editor_components.mako

  • 0
  • 0
  • 8
  • 0
  • 8
Description From Last Updated
  1. Nice one!

  2. placeholder? If so, perhaps cleaner to add a placeholder parameter

  3. This should be fixed, I recently adjusted the worker to always add it to tables

  4. .done instead of .always, if clean-up is needed use .fail

  5. should be rejected and fail if not an existing column, although could be wrong, but, if wrong we should fix in the CatalogEntry and reject there. Did you see this happen (when not using .always)?

Review request changed

Status: Closed (submitted)

  1. Nice!

    Note: the number input is a bit too large now? (e.g. with float of smallint)

    Add date/timestamp calendar (example in Oozie Coordinator submit) btw?