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

        Feng Peng created issue -
        Feng Peng made changes -
        Field Original Value New Value
        Attachment HIVE-3398_serde_empty_struct.patch [ 12542038 ]
        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.
        Feng Peng made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Fix Version/s 0.9.0 [ 12317742 ]
        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?
        Ashutosh Chauhan made changes -
        Status Patch Available [ 10002 ] Open [ 1 ]
        Fix Version/s 0.9.0 [ 12317742 ]
        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:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:

              Development