Uploaded image for project: 'Hue'
  1. Hue
  2. HUE-3063

[pig] Syntax highlight mistake around /* */

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.7.0
    • Fix Version/s: 3.10.0
    • Component/s: con.pig
    • Labels:
      None
    • Environment:

      CDH 5.4.8

      Description

      Enter this in the Pig query editor:

      fs -mkdir /tmp/foo
      fs -touchz /tmp/foo/bar
      fs -rm /tmp/foo/*
      fs -touchz /tmp/comment
      */

      Actual behavior: Pig removes the files /tmp/foo/* (being /tmp/foo/bar only at the time), then creates /tmp/comment, then fails on */ being a syntax error.

      Hue's Pig editor suggests: Pig will try to remove the file /tmp/foo, and then there's some comment enclosed by /* */.

      Apparently /* doesn't start a comment if preceded by a letter-like symbol (and presumably something similar might go for */ as well). Hue should highlight the syntax accordingly.

        Attachments

          Activity

            People

            • Assignee:
              enricoberti Enrico Berti
              Reporter:
              egmont Egmont Koblinger
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: