Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-7131

set() and toString Methods of the org.apache.hadoop.io.Text class does not include the root exception, in the wrapping RuntimeException.

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 0.20.1, 0.20.2
    • Fix Version/s: 0.23.0
    • Component/s: io
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      In below code snippets, we can include e, instead of e.toString(), so that caller can get complete trace.

      1)
      /** Set to contain the contents of a string.
      */
      public void set(String string) {
      try

      { ByteBuffer bb = encode(string, true); bytes = bb.array(); length = bb.limit(); }

      catch(CharacterCodingException e)

      { throw new RuntimeException("Should not have happened ",e.toString()); }
      }
      2)
      public String toString() {
      try { return decode(bytes, 0, length); } catch (CharacterCodingException e) { throw new RuntimeException("Should not have happened ",e.toString()); }

      }

        Attachments

        1. HADOOP-7131.patch
          0.8 kB
          Uma Maheswara Rao G

          Activity

            People

            • Assignee:
              umamaheswararao Uma Maheswara Rao G
              Reporter:
              umamaheswararao Uma Maheswara Rao G
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: