Uploaded image for project: 'CDAP'
  1. CDAP
  2. CDAP-6046

CLI syntax for "set preferences" is counter-intuitive

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.4.1
    • Fix Version/s: 4.1.0
    • Component/s: CLI
    • Labels:
    • Release Notes:
      Added an easier, additional syntax for the CDAP CLI set/get/load/delete <type> preferences commands, with the preferences at the end of the syntax, such as "set workflow preferences MyApp.My.WF 'a=b c=d'".
    • Rank:
      1|hzzdg7:

      Description

      The current syntax is: set preferences workflow "a=1 b=2" MyApp.MyWorkflow

      It could not be less intuitive, The keyword "workflow" should be followed by the workflow name. That will also allow for auto-completion. Also, the preferences themselves are kind of a vararg, which should always be at the end.

      It should be: set preferences for workflow MyApp.MyWorkflow "a=1 b=2"

      Finally, the help says:

      set preferences workflow <runtime-args> <app-id.workflow-id>
          Sets the preferences of a workflow. '<runtime-args>' is specified in the
          format 'key1=v1 key2=v2'.
      

      where does runtime arguments come from in this context. It should be preferences.

        Attachments

          Activity

            People

            • Assignee:
              jliu Joe Liu
              Reporter:
              andreas Andreas Neumann
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: