ICEpush
  1. ICEpush
  2. PUSH-26

Log a warning when pushing to a non-existent group

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Alpha2
    • Fix Version/s: 2.0-Alpha3
    • Component/s: Push Library
    • Labels:
      None
    • Environment:
      n/a

      Description

      It will be very useful in development to log a warning when PushContext.push(String group) is called for a non-existent group. Currently, the group name is treated as a single pushId if the group does not exist, but we could still check if that pushId exists, and if not, log a warning. This will improve the monitoring capabilities of an ICEpush application.

      Note that in a clustered environment a node may have no knowledge of a particular group, and a warning might be erroneously logged.

        Activity

        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P2
        Philip Breau made changes -
        Comment [ Is that warning from auction occurring because of the older icepush.jar version currently used in Glimmer? Can't we just check in the PushGroupManager impl like so:
        ]
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Mircea Toma made changes -
        Fix Version/s 1.0-Alpha3 [ 10224 ]
        Fix Version/s 1.0-Alpha2 [ 10223 ]
        Affects Version/s 1.0-Alpha2 [ 10223 ]
        Affects Version/s 1.0-Alpha3 [ 10224 ]
        Mircea Toma made changes -
        Affects Version/s 1.0-Alpha3 [ 10224 ]
        Affects Version/s 1.0-Alpha2 [ 10223 ]
        Deryk Sinotte made changes -
        Field Original Value New Value
        Summary Log a warning when pusing to a non-existent group Log a warning when pushing to a non-existent group
        Fix Version/s 1.0-Alpha2 [ 10223 ]
        Assignee Priority P2
        Affects Version/s 1.0-Alpha2 [ 10223 ]
        Assignee Mircea Toma [ mircea.toma ]
        Philip Breau created issue -

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Philip Breau
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: