HBase
  1. HBase
  2. HBASE-11112

PerformanceEvaluation should document --multiGet option on its printUsage.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.98.3
    • Fix Version/s: 0.99.0
    • Component/s: documentation, Performance
    • Labels:
      None
    1. HBASE-11112-v0-trunk.patch
      1 kB
      Jean-Marc Spaggiari
    2. HBASE-11112-v1-trunk.patch
      0.9 kB
      Jean-Marc Spaggiari

      Activity

      Hide
      Enis Soztutar added a comment -

      Closing this issue after 0.99.0 release.

      Show
      Enis Soztutar added a comment - Closing this issue after 0.99.0 release.
      Hide
      Hudson added a comment -

      ABORTED: Integrated in HBase-TRUNK #5132 (See https://builds.apache.org/job/HBase-TRUNK/5132/)
      HBASE-11112 PerformanceEvaluation should document --multiGet option on its printUsage (Jean-Marc Spaggiari) (ndimiduk: rev 1594757)

      • /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/PerformanceEvaluation.java
      Show
      Hudson added a comment - ABORTED: Integrated in HBase-TRUNK #5132 (See https://builds.apache.org/job/HBase-TRUNK/5132/ ) HBASE-11112 PerformanceEvaluation should document --multiGet option on its printUsage (Jean-Marc Spaggiari) (ndimiduk: rev 1594757) /hbase/trunk/hbase-server/src/test/java/org/apache/hadoop/hbase/PerformanceEvaluation.java
      Hide
      Nick Dimiduk added a comment -

      +1, with the nit that the line length is > 100 chars.

      I'm not sure where buildBot has gone off to, so I've cleaned up the line length and committed. Thanks for the patch, Jean-Marc Spaggiari!

      Show
      Nick Dimiduk added a comment - +1, with the nit that the line length is > 100 chars. I'm not sure where buildBot has gone off to, so I've cleaned up the line length and committed. Thanks for the patch, Jean-Marc Spaggiari !
      Hide
      Jean-Marc Spaggiari added a comment -

      So more something like that? Pretty small

      Show
      Jean-Marc Spaggiari added a comment - So more something like that? Pretty small
      Hide
      Nick Dimiduk added a comment -

      I caught this and slipped it in with HBASE-10791, but that's on the 10070 branch that isn't finished. We can commit this one and deal with the merge on the other branch later; fine by me.

      My only nit is that we don't need the --multiGet in the usage information; extended options aren't exposed there.

      Show
      Nick Dimiduk added a comment - I caught this and slipped it in with HBASE-10791 , but that's on the 10070 branch that isn't finished. We can commit this one and deal with the merge on the other branch later; fine by me. My only nit is that we don't need the --multiGet in the usage information; extended options aren't exposed there.
      Hide
      Jean-Marc Spaggiari added a comment -

      Pretty simple patch.
      Nick Dimiduk Fine with that?

      Show
      Jean-Marc Spaggiari added a comment - Pretty simple patch. Nick Dimiduk Fine with that?

        People

        • Assignee:
          Jean-Marc Spaggiari
          Reporter:
          Jean-Marc Spaggiari
        • Votes:
          0 Vote for this issue
          Watchers:
          4 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development