Hive
  1. Hive
  2. HIVE-1749

ExecMapper and ExecReducer: reduce function calls to l4j.isInfoEnabled()

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.7.0
    • Component/s: Query Processor
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Calling l4j.isInfoEnabled() is more expensive than we thought. By eliminating this function call, we can save 1% - 3% CPU time, according to the profiliing.

        Activity

        Hide
        Namit Jain added a comment -

        Committed. Thanks Siyng

        Show
        Namit Jain added a comment - Committed. Thanks Siyng
        Hide
        Namit Jain added a comment -

        +1

        Show
        Namit Jain added a comment - +1
        Hide
        Siying Dong added a comment -

        From ExecMapper and ExecReducer, save result of l4j.isInfoEnabled() in configure()

        Show
        Siying Dong added a comment - From ExecMapper and ExecReducer, save result of l4j.isInfoEnabled() in configure()

          People

          • Assignee:
            Siying Dong
            Reporter:
            Siying Dong
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development