Details

    • Type: Sub-task Sub-task
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      This is a sub-task for Syntax Errors and use cases for it.

      Having PARALLEL in wrong places is confusing for many users. I just saw somebody putting it after STORE. Adding it to FILTER is very common as well.

        Issue Links

          Activity

          Hide
          Olga Natkovich added a comment -

          We want to address error handling as part of parser re-work

          Show
          Olga Natkovich added a comment - We want to address error handling as part of parser re-work
          Hide
          Alan Gates added a comment -

          Since part of the current language specification is that parallel is allowed after any operator, we cannot simply remove it. We could add warnings to tell users when parallel isn't doing anything for them, and deprecate it's usage for those operators.

          But even then I'm not sure how useful this is. Some operators will sometimes make use of parallel and sometimes not, depending on implementation (e.g. join would use it for hash join, but not for merge join). So users will always have to know when parallel is and isn't helping them.

          Show
          Alan Gates added a comment - Since part of the current language specification is that parallel is allowed after any operator, we cannot simply remove it. We could add warnings to tell users when parallel isn't doing anything for them, and deprecate it's usage for those operators. But even then I'm not sure how useful this is. Some operators will sometimes make use of parallel and sometimes not, depending on implementation (e.g. join would use it for hash join, but not for merge join). So users will always have to know when parallel is and isn't helping them.
          Hide
          Olga Natkovich added a comment -

          Delaying till 10 to let current parser changes to stabilize

          Show
          Olga Natkovich added a comment - Delaying till 10 to let current parser changes to stabilize

            People

            • Assignee:
              Xuefu Zhang
              Reporter:
              Amir Youssefi
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:

                Development