Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.0.0
    • Fix Version/s: 1.1.0
    • Component/s: blueprints
    • Labels:
      None

      Description

      Extract samplers library from BigPetStore data generator.

        Activity

        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user rnowling commented on the pull request:

        https://github.com/apache/bigtop/pull/34#issuecomment-134592298

        Closing due to conflicts. Will resubmit as patch.

        Show
        githubbot ASF GitHub Bot added a comment - Github user rnowling commented on the pull request: https://github.com/apache/bigtop/pull/34#issuecomment-134592298 Closing due to conflicts. Will resubmit as patch.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user rnowling closed the pull request at:

        https://github.com/apache/bigtop/pull/34

        Show
        githubbot ASF GitHub Bot added a comment - Github user rnowling closed the pull request at: https://github.com/apache/bigtop/pull/34
        Hide
        rnowling RJ Nowling added a comment -

        This patch reorganizes the bigpetstore-data-generator packages and moves the samplers into a separate gradle projects.

        The bigpetstore-data-generator is now under org.apache.bigtop.datagenerators.bigpetstore and the samplers are under org.apache.bigtop.datagenerators.samplers.

        Show
        rnowling RJ Nowling added a comment - This patch reorganizes the bigpetstore-data-generator packages and moves the samplers into a separate gradle projects. The bigpetstore-data-generator is now under org.apache.bigtop.datagenerators.bigpetstore and the samplers are under org.apache.bigtop.datagenerators.samplers .
        Hide
        rnowling RJ Nowling added a comment -

        jay vyas When you get a chance, can you review? Thanks!

        Show
        rnowling RJ Nowling added a comment - jay vyas When you get a chance, can you review? Thanks!
        Hide
        jayunit100 jay vyas added a comment -

        +1

        Show
        jayunit100 jay vyas added a comment - +1
        Hide
        rnowling RJ Nowling added a comment -

        Thanks! Merged!

        Show
        rnowling RJ Nowling added a comment - Thanks! Merged!

          People

          • Assignee:
            rnowling RJ Nowling
            Reporter:
            rnowling RJ Nowling
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development