Uploaded image for project: 'Ignite'
  1. Ignite
  2. IGNITE-12464 Metrics for the Java Services.
  3. IGNITE-16400

Deprecate direct references to local services. Fix service proxy.

    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Resolved
    • 2.12, 2.11.1
    • 2.13
    • None
    • Deprecated IgniteServices#service(String) and IgniteServices#services(String).
    • Docs Required, Release Notes Required

    Description

      As discussed in the threads related to the main ticket #IGNITE-12464, we deprecate references to local services like

      IgniteServices#service()
      IgniteServices#services()
      

      and fix behavior of the proxies like

      serviceProxy(String name, Class<? super T> svcItf, boolean sticky)
      

      so that a proxy is given every time even for local services.

      Reasons in short:

      • Direct references to local services corrupt the service statistics.
      • Direct references to local services bring no real optimization.
      • `serviceProxy()` says 'proxy', '@return Proxy over service'. It should return a proxy and do not variate behavior. Especially depending on user-side settings

      Also, the documentation of service metrics and limitations or local services is going to get improved.

      Attachments

        Issue Links

          Activity

            People

              vladsz83 Vladimir Steshin
              vladsz83 Vladimir Steshin
              Maksim Timonin Maksim Timonin
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 10m
                  10m