Uploaded image for project: 'ProtoStream'
  1. ProtoStream
  2. IPROTO-98

Schema parser should recognize and enforce the 'reserved' statement

    XMLWordPrintable

Details

    • Enhancement
    • Resolution: Done
    • Major
    • 5.0.0.Dev01
    • None
    • None
    • DataGrid Sprint #30

    Description

      This is a continuation of IPROTO-11

      See
      https://developers.google.com/protocol-buffers/docs/proto#reserved
      https://developers.google.com/protocol-buffers/docs/proto3#reserved

      The 'reserved' keyword was added in protobuf-java v3.0.0-alpha-3, which came years after the protoparser 4.0.3 we use so this cannot work unless we upgrade/replace protoparser.

      Attachments

        Issue Links

          is duplicated by

          Bug - A problem that impairs or prevents the functions of the product. IPROTO-210 @ProtoReserved will create comments instead of protobuf types

          • Minor - To be worked after urgent, high, and medium priorities are resolved. This term is chosen when the severity of the issue is low, or the complexity or effort to correct it may be higher, relatively speaking. For low priority issues, known workarounds exist or are not needed due to the trivial effort needed to address the issue.
          • Closed

          Enhancement - An enhancement to or refactoring of existing functionality that is not configurable by an end user (typically a change made by an internal team that affects users) IPROTO-117 Proto parser must check and enforce reserved fields

          • Major - To be worked after higher priority work items are addressed. When the severity is high and the effort to change it is low to moderate, this term is chosen. Issues in this category likely have an existing workaround but implementation or execution may be non-trivial.
          • Closed

          Activity

            People

              ttarrant@redhat.com Tristan Tarrant
              anistor Adrian Nistor (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: