Hive
  1. Hive
  2. HIVE-3398

Hive serde should support empty struct

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 0.9.0
    • Fix Version/s: None
    • Labels:
      None

      Description

      Right now TypeInfoUtils expects at least one field in a STRUCT, which is not always true, e.g., empty struct is allowed in Thrift. We should modify TypeInfoUtils so that empty struct can be correctly processed.

        Activity

        Hide
        Feng Peng added a comment -

        Patch attached.

        Show
        Feng Peng added a comment - Patch attached.
        Hide
        Feng Peng added a comment -

        The actual change is pretty simple. Most of the contents in the patch are caused by the change of the test thrift struct.

        Show
        Feng Peng added a comment - The actual change is pretty simple. Most of the contents in the patch are caused by the change of the test thrift struct.
        Hide
        Feng Peng added a comment -

        Posted patch on reviewboard: https://reviews.apache.org/r/7042/.

        Show
        Feng Peng added a comment - Posted patch on reviewboard: https://reviews.apache.org/r/7042/ .
        Hide
        Ashutosh Chauhan added a comment -

        Patch doesn't apply cleanly anymore. Feng can you refresh the patch?

        Show
        Ashutosh Chauhan added a comment - Patch doesn't apply cleanly anymore. Feng can you refresh the patch?
        Hide
        Feng Peng added a comment -

        Sure, thanks for checking it out!

        Show
        Feng Peng added a comment - Sure, thanks for checking it out!
        Hide
        Allan Shoup added a comment -

        Any updates on applying this patch?

        Show
        Allan Shoup added a comment - Any updates on applying this patch?

          People

          • Assignee:
            Unassigned
            Reporter:
            Feng Peng
          • Votes:
            1 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:

              Development