Details

    • Type: Bug
    • Status: Patch Available
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: 0.14.0
    • Component/s: ClientDSL
    • Labels:

      Description

      If you want to insert some data in HBase the only available method is store(), which allow you to put only one row in a table. That means if you want to put 10k rows, you have to do 10k HTTP calls… The HBase REST API provides a multi-put

      1. KNOX-808.patch
        8 kB
        Vincent Devillers

        Activity

        Hide
        lmccay Larry McCay added a comment -

        Vincent Devillers - Do you get this note now that you changed your email?

        Show
        lmccay Larry McCay added a comment - Vincent Devillers - Do you get this note now that you changed your email?
        Hide
        smore Sandeep More added a comment -

        Hello Vincent Devillers, would be able to submit a new patch.

        Best,
        Sandeep

        Show
        smore Sandeep More added a comment - Hello Vincent Devillers , would be able to submit a new patch. Best, Sandeep
        Hide
        lmccay Larry McCay added a comment -

        After a quick review, I am going to push this out of 0.13.0.

        We need to consider refactoring the StoreRow and StoreRows to use common code and the patch hardcodes Row and Cell, CellSet, etc whereas StoreRow is using constants.

        The refactoring may actually make resolving those hardcoded strings a moot point.
        Since this needs some considerable work and an apache license - let's move it out as to not block 0.13.0.

        Show
        lmccay Larry McCay added a comment - After a quick review, I am going to push this out of 0.13.0. We need to consider refactoring the StoreRow and StoreRows to use common code and the patch hardcodes Row and Cell, CellSet, etc whereas StoreRow is using constants. The refactoring may actually make resolving those hardcoded strings a moot point. Since this needs some considerable work and an apache license - let's move it out as to not block 0.13.0.
        Hide
        smore Sandeep More added a comment -

        Thanks for the patch Vincent Devillers and sorry for the delay in getting to it.
        I was trying to test the patch, it appears that the class StoreRows does not have Apache License which causes the builds to fail.
        Few more things, I would also appreciate if you could provide an unit tests and an example usage.

        Again thanks for the contribution !

        Show
        smore Sandeep More added a comment - Thanks for the patch Vincent Devillers and sorry for the delay in getting to it. I was trying to test the patch, it appears that the class StoreRows does not have Apache License which causes the builds to fail. Few more things, I would also appreciate if you could provide an unit tests and an example usage. Again thanks for the contribution !
        Hide
        treydone Vincent Devillers added a comment -

        Sumit Gupta Yes, it makes sense, I've updated the patch

        Show
        treydone Vincent Devillers added a comment - Sumit Gupta Yes, it makes sense, I've updated the patch
        Hide
        lmccay Larry McCay added a comment -

        Moving this out to 0.13.0 so that we can decide on the best design for the API extension.

        Show
        lmccay Larry McCay added a comment - Moving this out to 0.13.0 so that we can decide on the best design for the API extension.
        Hide
        sumit.gupta Sumit Gupta added a comment -

        Vincent Devillers, does this make more sense as a 'Rows' class instead of multiple rows getting put on a Row class? The pattern for 'put' could seem strange here. What do you think? Any examples of usage would also be appreciated.

        Show
        sumit.gupta Sumit Gupta added a comment - Vincent Devillers , does this make more sense as a 'Rows' class instead of multiple rows getting put on a Row class? The pattern for 'put' could seem strange here. What do you think? Any examples of usage would also be appreciated.

          People

          • Assignee:
            treydone Vincent Devillers
            Reporter:
            treydone Vincent Devillers
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:

              Development