Uploaded image for project: 'Infinispan'
  1. Infinispan
  2. ISPN-9861

Marshallers for remote querying should be public

    XMLWordPrintable

Details

    • Task
    • Resolution: Won't Do
    • Major
    • 10.0.0.Alpha3
    • None
    • Remote Querying
    • None

    Description

      There are a number of marshallers that are registered to enable remote querying. These classes are currently only enabled via 

      MarshallerRegistration#registerMarshallers

      method. These classes should be public to allow other methods if needed.

      The marshallers used are currently:

            ctx.registerMarshaller(new QueryRequest.NamedParameter.Marshaller());
      ctx.registerMarshaller(new QueryRequest.Marshaller());
      ctx.registerMarshaller(new QueryResponse.Marshaller());
      ctx.registerMarshaller(new FilterResult.Marshaller());
      ctx.registerMarshaller(new ContinuousQueryResult.ResultType.Marshaller());
      ctx.registerMarshaller(new ContinuousQueryResult.Marshaller());

      Note that the classes that are marshalled by these are all public, so there shouldn't be any harm in changing the visibility.

      Attachments

        Issue Links

          is related to

          Enhancement - An enhancement or refactoring of existing functionality ISPN-9862 Split org.infinispan.query.remote.client.MarshallerRegistration.registerMarshallers

          • 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

            Public project attachment banner

              context keys: [headless, issue, helper, isAsynchronousRequest, project, action, user]
              current Project key: ISPN

              People

                wburns@redhat.com Will Burns
                wburns@redhat.com Will Burns
                Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: