Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Won't Fix
    • None
    • None
    • None
    • None

    Description

      When you see code like this:
      first = true;
      for (int k = 0; k < columnSize; k++) {
      String newColName = i + "VALUE" + k; // any name, it does not matter.
      + newColName = i + "VALUE" + k; // any name, it does not matter.
      if (!first)
      Unknown macro:

      { - valueColNames = valueColNames + ","; - valueColTypes = valueColTypes + ","; + valueColNames.append(","); + valueColTypes.append(","); }

      valueColNames = valueColNames + newColName;
      valueColTypes = valueColTypes + valueCols.get(k).getTypeString();
      + valueColNames.append(newColName);
      + valueColTypes.append(valueCols.get(k).getTypeString());
      first = false;
      Can you replace it with StringUtil.join()
      I have seen this about 4 places in hive.

      Attachments

        Activity

          People

            benjamin.jakobus Benjamin Jakobus
            benjamin.jakobus Benjamin Jakobus
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: