HBase
  1. HBase
  2. HBASE-438

XMLOutputter state should be initialized.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 0.1.0
    • Fix Version/s: 0.2.0
    • Component/s: util
    • Labels:
      None

      Description

      bash-3.00# bin/hbase shell --html
      HQL, 0.2.0-dev version.
      Copyright (c) 2008 by udanax, licensed to Apache Software Foundation.
      Type 'help;' for usage.
      
      hql > show tables;
      <table>
       <tr>
        <th>
      Name
        </th>
        <th>
      Descriptor
        </th>
       </tr>
       <tr>
        <td>
      a
        </td>
        <td>
      name: a, families: {b:={name: b, max versions: 3, compression: NONE, in memory: false, block cache enabled: false, max length: 2147483647, bloom filter: none}}
        </td>
       </tr>
      </table>
      1 table(s) in set. (0.21 sec)
      hql > show tables;
      Exception in thread "main" java.lang.IllegalStateException: getState() == DOCUMENT_ENDED
              at org.znerd.xmlenc.XMLOutputter.startTag(Unknown Source)
              at org.apache.hadoop.hbase.hql.formatter.HtmlTableFormatter.header(HtmlTableFormatter.java:125)
              at org.apache.hadoop.hbase.hql.ShowCommand.execute(ShowCommand.java:66)
              at org.apache.hadoop.hbase.hql.HQLClient.executeQuery(HQLClient.java:68)        at org.apache.hadoop.hbase.Shell.main(Shell.java:114)
      bash-3.00#
      
      1. 438.patch
        0.8 kB
        Edward J. Yoon

        Activity

        Hide
        Edward J. Yoon added a comment -

        Submitting.

        Show
        Edward J. Yoon added a comment - Submitting.
        Hide
        Bryan Duxbury added a comment -

        Tests pass locally for me. Patch seems pretty simple. +1

        Show
        Bryan Duxbury added a comment - Tests pass locally for me. Patch seems pretty simple. +1
        Hide
        Edward J. Yoon added a comment -

        Thanks for your review, bryan.
        Please commit it.

        Show
        Edward J. Yoon added a comment - Thanks for your review, bryan. Please commit it.
        Hide
        Jim Kellerman added a comment -

        Patch looks ok +1

        Show
        Jim Kellerman added a comment - Patch looks ok +1
        Hide
        Bryan Duxbury added a comment -

        Committed.

        Show
        Bryan Duxbury added a comment - Committed.

          People

          • Assignee:
            Edward J. Yoon
            Reporter:
            Edward J. Yoon
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development