Uploaded image for project: 'Hive'
  1. Hive
  2. HIVE-21661

Add option to check q file syntax before execution

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0.0
    • Component/s: Testing Infrastructure
    • Labels:
      None

      Description

      Currently q file tests are executed line-by-line. This could lead to problems, e.g. when a q file has some long running queries and the last line contains a syntax error. In this case everything before the wrong line will be executed first, which could take a lot of time and only then will the test fail due to the syntax error.

      I propose a simple syntax checker that will check the q file for errors before executing any statements. This check can will be turned off by default and can be enabled with the following option: 

      -Dtest.check.syntax=true

        Attachments

        1. HIVE-21661.1.patch
          7 kB
          Bruno Pusztahazi
        2. HIVE-21661.2.patch
          8 kB
          Bruno Pusztahazi

          Activity

            People

            • Assignee:
              bpusztahazi Bruno Pusztahazi
              Reporter:
              bpusztahazi Bruno Pusztahazi
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: