Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.92.2
    • Component/s: master
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      This JIRA continues the effort from HBASE-5179. Starting with Stack's comments about patches for 0.92 and TRUNK:

      Reviewing 0.92v17

      isDeadServerInProgress is a new public method in ServerManager but it does not seem to be used anywhere.

      Does isDeadRootServerInProgress need to be public? Ditto for meta version.

      This method param names are not right 'definitiveRootServer'; what is meant by definitive? Do they need this qualifier?

      Is there anything in place to stop us expiring a server twice if its carrying root and meta?

      What is difference between asking assignment manager isCarryingRoot and this variable that is passed in? Should be doc'd at least. Ditto for meta.

      I think I've asked for this a few times - onlineServers needs to be explained... either in javadoc or in comment. This is the param passed into joinCluster. How does it arise? I think I know but am unsure. God love the poor noob that comes awandering this code trying to make sense of it all.

      It looks like we get the list by trawling zk for regionserver znodes that have not checked in. Don't we do this operation earlier in master setup? Are we doing it again here?

      Though distributed split log is configured, we will do in master single process splitting under some conditions with this patch. Its not explained in code why we would do this. Why do we think master log splitting 'high priority' when it could very well be slower. Should we only go this route if distributed splitting is not going on. Do we know if concurrent distributed log splitting and master splitting works?

      Why would we have dead servers in progress here in master startup? Because a servershutdownhandler fired?

      This patch is different to the patch for 0.90. Should go into trunk first with tests, then 0.92. Should it be in this issue? This issue is really hard to follow now. Maybe this issue is for 0.90.x and new issue for more work on this trunk patch?

      This patch needs to have the v18 differences applied.

        Attachments

        1. HBASE-5270-92v11.patch
          27 kB
          chunhui shen
        2. HBASE-5270v11.patch
          32 kB
          chunhui shen
        3. hbase-5270v10.patch
          32 kB
          chunhui shen
        4. hbase-5270v9.patch
          28 kB
          chunhui shen
        5. hbase-5270v8.patch
          28 kB
          chunhui shen
        6. hbase-5270v7.patch
          29 kB
          chunhui shen
        7. hbase-5270v6.patch
          30 kB
          chunhui shen
        8. hbase-5270v5.patch
          30 kB
          chunhui shen
        9. hbase-5270v4.patch
          30 kB
          chunhui shen
        10. 5270-90v3.patch
          39 kB
          chunhui shen
        11. 5270-90v2.patch
          38 kB
          chunhui shen
        12. 5270-90-testcasev2.patch
          15 kB
          chunhui shen
        13. 5270-testcasev2.patch
          15 kB
          chunhui shen
        14. hbase-5270v2.patch
          35 kB
          chunhui shen
        15. sampletest.txt
          6 kB
          stack
        16. 5270-90-testcase.patch
          9 kB
          chunhui shen
        17. 5270-90.patch
          31 kB
          chunhui shen
        18. 5270-testcase.patch
          10 kB
          chunhui shen
        19. hbase-5270.patch
          28 kB
          chunhui shen

          Issue Links

            Activity

              People

              • Assignee:
                zjushch chunhui shen
                Reporter:
                zhihyu@ebaysf.com Ted Yu
              • Votes:
                0 Vote for this issue
                Watchers:
                8 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: