HBase
  1. HBase
  2. HBASE-5948

Deprecate and remove the Avro gateway

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.95.2
    • Fix Version/s: 0.95.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Deprecate the Avro gateway in 0.94. Remove in 0.96. Made a blocker against that release.

      1. HBASE-5948-1.patch
        2 kB
        Elliott Clark
      2. HBASE-5948-0.patch
        2 kB
        Elliott Clark

        Issue Links

          Activity

          Hide
          Andrew Purtell added a comment -

          This is falling further and further behind and no owner has materialized. Stack and I were talking about this the other day. One option is to replace it with a completed HBASE-2557.

          Show
          Andrew Purtell added a comment - This is falling further and further behind and no owner has materialized. Stack and I were talking about this the other day. One option is to replace it with a completed HBASE-2557 .
          Hide
          stack added a comment -

          +1

          Show
          stack added a comment - +1
          Hide
          Todd Lipcon added a comment -

          +1, let's move it to github so anyone who wants to pick it up can do so.

          Show
          Todd Lipcon added a comment - +1, let's move it to github so anyone who wants to pick it up can do so.
          Hide
          Elliott Clark added a comment -

          Patch to deprecate. Includes a small note in the api.

          Show
          Elliott Clark added a comment - Patch to deprecate. Includes a small note in the api.
          Hide
          Andrew Purtell added a comment -

          +1 Thanks Elliott.

          Show
          Andrew Purtell added a comment - +1 Thanks Elliott.
          Hide
          Elliott Clark added a comment -

          reabse

          Show
          Elliott Clark added a comment - reabse
          Hide
          stack added a comment -

          Committed to trunk. Thanks for the patch Elliott

          Show
          stack added a comment - Committed to trunk. Thanks for the patch Elliott
          Hide
          Hudson added a comment -

          Integrated in HBase-TRUNK #2947 (See https://builds.apache.org/job/HBase-TRUNK/2947/)
          HBASE-5948 Deprecate and remove the Avro gateway (Revision 1344029)

          Result = FAILURE
          stack :
          Files :

          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/avro/AvroServer.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/avro/AvroUtil.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/avro/package.html
          Show
          Hudson added a comment - Integrated in HBase-TRUNK #2947 (See https://builds.apache.org/job/HBase-TRUNK/2947/ ) HBASE-5948 Deprecate and remove the Avro gateway (Revision 1344029) Result = FAILURE stack : Files : /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/avro/AvroServer.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/avro/AvroUtil.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/avro/package.html
          Hide
          Hudson added a comment -

          Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #31 (See https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/31/)
          HBASE-5948 Deprecate and remove the Avro gateway (Revision 1344029)

          Result = FAILURE
          stack :
          Files :

          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/avro/AvroServer.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/avro/AvroUtil.java
          • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/avro/package.html
          Show
          Hudson added a comment - Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #31 (See https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/31/ ) HBASE-5948 Deprecate and remove the Avro gateway (Revision 1344029) Result = FAILURE stack : Files : /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/avro/AvroServer.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/avro/AvroUtil.java /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/avro/package.html
          Hide
          David S. Wang added a comment -

          The patch seems to deprecate in 0.96 without removing it. The original goal was to deprecate in 0.94 and remove in 0.96, correct?

          Show
          David S. Wang added a comment - The patch seems to deprecate in 0.96 without removing it. The original goal was to deprecate in 0.94 and remove in 0.96, correct?
          Hide
          David S. Wang added a comment -

          I should mention that I can provide a patch on top of this one to do the removal for 0.96; then a committer can just backport this patch to 0.94 to fulfill the original intent. I'll file that JIRA with the patch shortly unless I hear objections.

          Show
          David S. Wang added a comment - I should mention that I can provide a patch on top of this one to do the removal for 0.96; then a committer can just backport this patch to 0.94 to fulfill the original intent. I'll file that JIRA with the patch shortly unless I hear objections.
          Hide
          Elliott Clark added a comment -

          HBASE-6553 did the actual removal. I believe stack committed the deprecation into 0.94 branch only

          Show
          Elliott Clark added a comment - HBASE-6553 did the actual removal. I believe stack committed the deprecation into 0.94 branch only
          Hide
          David S. Wang added a comment -

          That's what I get for looking at an outdated tree, and for having too many trees in general. Sorry for the spam.

          Show
          David S. Wang added a comment - That's what I get for looking at an outdated tree, and for having too many trees in general. Sorry for the spam.
          Hide
          stack added a comment -

          Marking closed.

          Show
          stack added a comment - Marking closed.

            People

            • Assignee:
              Elliott Clark
              Reporter:
              Andrew Purtell
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development