Hive
  1. Hive
  2. HIVE-576 complete jdbc driver
  3. HIVE-2158

add the HivePreparedStatement implementation based on current HIVE supported data-type

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.6.0, 0.7.0, 0.8.0
    • Fix Version/s: 0.7.1, 0.8.0
    • Component/s: JDBC
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Activity

      Hide
      Yuanjun Li added a comment -

      All patches have been submitted

      Show
      Yuanjun Li added a comment - All patches have been submitted
      Hide
      jiraposter@reviews.apache.org added a comment -

      -----------------------------------------------------------
      This is an automatically generated e-mail. To reply, visit:
      https://reviews.apache.org/r/900/
      -----------------------------------------------------------

      Review request for hive.

      Summary
      -------

      Review for HIVe-2158.

      This addresses bug HIVE-2158.
      https://issues.apache.org/jira/browse/HIVE-2158

      Diffs


      jdbc/src/java/org/apache/hadoop/hive/jdbc/HivePreparedStatement.java e27ec66
      jdbc/src/test/org/apache/hadoop/hive/jdbc/TestJdbcDriver.java 3633326

      Diff: https://reviews.apache.org/r/900/diff

      Testing
      -------

      Thanks,

      Carl

      Show
      jiraposter@reviews.apache.org added a comment - ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/900/ ----------------------------------------------------------- Review request for hive. Summary ------- Review for HIVe-2158. This addresses bug HIVE-2158 . https://issues.apache.org/jira/browse/HIVE-2158 Diffs jdbc/src/java/org/apache/hadoop/hive/jdbc/HivePreparedStatement.java e27ec66 jdbc/src/test/org/apache/hadoop/hive/jdbc/TestJdbcDriver.java 3633326 Diff: https://reviews.apache.org/r/900/diff Testing ------- Thanks, Carl
      Hide
      Carl Steinbach added a comment -

      +1.

      Show
      Carl Steinbach added a comment - +1.
      Hide
      Carl Steinbach added a comment -

      Committed to trunk and branch-0.7. Thanks Yuanjun!

      Show
      Carl Steinbach added a comment - Committed to trunk and branch-0.7. Thanks Yuanjun!
      Hide
      Yuanjun Li added a comment -

      Well, the Jasperreport can work well directly with Hive official release now.

      Show
      Yuanjun Li added a comment - Well, the Jasperreport can work well directly with Hive official release now.
      Hide
      Hudson added a comment -

      Integrated in Hive-trunk-h0.21 #790 (See https://builds.apache.org/job/Hive-trunk-h0.21/790/)

      Show
      Hudson added a comment - Integrated in Hive-trunk-h0.21 #790 (See https://builds.apache.org/job/Hive-trunk-h0.21/790/ )
      Hide
      Ido Hadanny added a comment -

      setObject still implemented, and that's blocking spring's jdbcTemplate.execute(sql, args).
      If I want to add it, should I open a new issue, or add another patch here?

      Show
      Ido Hadanny added a comment - setObject still implemented, and that's blocking spring's jdbcTemplate.execute(sql, args). If I want to add it, should I open a new issue, or add another patch here?
      Hide
      Carl Steinbach added a comment -

      @Ido: please open a new ticket and post your patch there. Thanks!

      Show
      Carl Steinbach added a comment - @Ido: please open a new ticket and post your patch there. Thanks!

        People

        • Assignee:
          Yuanjun Li
          Reporter:
          Yuanjun Li
        • Votes:
          0 Vote for this issue
          Watchers:
          2 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Development