Uploaded image for project: 'HPCC'
  1. HPCC
  2. HPCC-22230

COMBINE documentation inaccurate

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 7.2.20
    • Component/s: Documentation
    • Labels:
      None

      Description

      The documentation for the COMBINE function states:

       

            COMBINE( leftrecset, rightrecset [, transform ][,LOCAL])

            LOCAL The LOCAL option is required when COMBINE is used on Thor (and implicit in hThor/Roxie)

       

      If I try to use LOCAL without the optional transform specified, I get a syntax error:

        

         Error:    Unknown identifier "LOCAL" (106, 35), 2167,

       

      If I don't specify LOCAL and run on Thor, i get a runtime error:

       

            Warning:  (106,9): error C4619: Thor currently only supports the local version of COMBINE (0, 0), 0,

       

      Looks like it is syntactically and operationally correct when the optional transform is supplied and LOCAL is specified when run on Thor, but then the documentation needs to reflect that.

       

      Examples:

       

      Works on Thor:  

       

            dOut := COMBINE(dInPrep, dPhones, combineThem(LEFT, RIGHT), LOCAL);

       

      Won’t syntax check:

       

            dOut := COMBINE(dInPrep, dPhones, LOCAL);

       

      Won’t run on Thor:

       

            dOut := COMBINE(dInPrep, dPhones);

       

       

      Jim DeFabia Gavin Halliday Richard Taylor

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                jamesdefabia Jim DeFabia
                Reporter:
                kev77log Kevin Logemann
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Due:
                  Created:
                  Updated:
                  Resolved: