Uploaded image for project: 'Pig'
  1. Pig
  2. PIG-5237

Fix DOT file parsing to enable DOT-based physical plan testing

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 0.16.0
    • 0.18.0
    • tools
    • None

    Attachments

      1. PIG-5237.2.patch
        23 kB
        Jakov Rabinovits
      2. PIG-5237.patch
        20 kB
        Jakov Rabinovits

      Issue Links

        Activity

          Fixed DOT graph grammar and modified DOT graph classes to support toString() operation and checking isomorphism. Test now uses the DOT graph classes.

          YaShock Jakov Rabinovits added a comment - Fixed DOT graph grammar and modified DOT graph classes to support toString() operation and checking isomorphism. Test now uses the DOT graph classes.
          szita Ádám Szita added a comment -

          YaShock patch looks good. Can you please reformat the code as per PIG-4604?

          szita Ádám Szita added a comment - YaShock patch looks good. Can you please reformat the code as per PIG-4604 ?
          szita Ádám Szita added a comment -

          +1 for PIG-5237.2.patch. It is now committed to trunk, thanks for the contribution YaShock!

          szita Ádám Szita added a comment - +1 for PIG-5237.2.patch . It is now committed to trunk, thanks for the contribution YaShock !

          People

            YaShock Jakov Rabinovits
            szita Ádám Szita
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: