[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-03-25 Thread jdu...@cloudbees.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 James Dumay commented on  JENKINS-41063  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
 Oh nice one   
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-03-25 Thread tom.fenne...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Tom FENNELLY closed an issue as Fixed  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Actually this was fixed back in Jan in v1.12 of the plugin. Not sure why it was not closed.  
 

  
 
 
 
 

 
 Jenkins /  JENKINS-41063  
 
 
  SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
Change By: 
 Tom FENNELLY  
 
 
Status: 
 Open Closed  
 
 
Resolution: 
 Fixed  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to 

[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-03-25 Thread jdu...@cloudbees.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 James Dumay commented on  JENKINS-41063  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
 We don't have this on the plan to fix before 1.0 however there is some work going on at the moment to make SSE work across sessions and it may fix this incidentally - Vivek Pandey have you run into this with your current project?  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-03-25 Thread o.v.nenas...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Oleg Nenashev commented on  JENKINS-41063  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
 James Dumay Tom FENNELLY IIRC you were going to fix it before the public Blue Ocean release. Were there any changes in the plan?  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-03-19 Thread jdu...@cloudbees.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 James Dumay updated an issue  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Jenkins /  JENKINS-41063  
 
 
  SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
Change By: 
 James Dumay  
 
 
Priority: 
 Critical Major  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.3.0#73011-sha1:3c73d0e)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-01-16 Thread tom.fenne...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Tom FENNELLY commented on  JENKINS-41063  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
 I don't plan on doing anything more on this until after we have more details of how this might be used in a clustering env. I'm not going to remove the Serializable interface marker because that will result in some Servlet containers barfing on it completely. Also, making some of the fields serializable is just a waste of time too imo, because there's a lot more to making it actually work in a clustered env - the backend event bus subscriptions need to be re-established too etc, or the client will just stop receiving events. If the sessions are not sticky and browsers are constantly being assigned to a replicated session on a different node, then I'd bet that will cause a litany of other ugly issues. TL;DR ... we need to figure out how clustering might work before doing a "real" fix for this. Anything else is just sweeping crap under the carpet.  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-01-16 Thread tom.fenne...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Tom FENNELLY updated an issue  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Jenkins /  JENKINS-41063  
 
 
  SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
Change By: 
 Tom FENNELLY  
 
 
Sprint: 
 tethys  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-01-16 Thread tom.fenne...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Tom FENNELLY updated an issue  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Jenkins /  JENKINS-41063  
 
 
  SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
Change By: 
 Tom FENNELLY  
 
 
Sprint: 
 tethys  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-01-16 Thread tom.fenne...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Tom FENNELLY commented on  JENKINS-41063  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
 I did a 1.12 release of the SSE Gateway Plugin that makes the EventDispatcher serializable so as not to raise errors in the session manager. Yes, it would not be deserializable but that doesn't matter for now.  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-01-16 Thread jdu...@cloudbees.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 James Dumay commented on  JENKINS-41063  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
 Can you send the ux team at CloudBees an email?  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-01-16 Thread o.v.nenas...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Oleg Nenashev commented on  JENKINS-41063  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
 James Dumay Previously I have provided links to my fixes to Tom, but something still needs to be applied to the plugin's upstream. Just making fields transient will formally work, but after that deserialized event dispatcher will not work at all. Hence I had a more complex fix (sorry, cannot provide it here since it's not OSS)  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-01-15 Thread jdu...@cloudbees.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 James Dumay commented on  JENKINS-41063  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
 Oleg Nenashev did you get a resolution here?  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-01-13 Thread tom.fenne...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Tom FENNELLY commented on  JENKINS-41063  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
 I'm fine with just making the relevant fields transient.  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-01-13 Thread o.v.nenas...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Oleg Nenashev commented on  JENKINS-41063  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
 I have a private patch for serializable EventDispatcher. Can share it with you though the code is not open-source. As a temporary fix I would be fine if you just remove the "implements Serializable" definition. It won't impact the default session manager. OTOH, it's not binary compatible.  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-01-13 Thread tom.fenne...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Tom FENNELLY commented on  JENKINS-41063  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
  Re: SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
 The SSE Gateway really only uses the servlet session as a way of "scoping" dispatchers to the browser sessions with which the are associated i.e. not as a means of "persisting" anything across sessions. So from a non clustered point of view, the easy fix here is to just make all the fields transient. That should at least stop the servlet container session manager from barfing. If it's a case that we ever want to make this clusterable, I think we'd need to first have a clear idea about the overall clustering architecture that it needs to work in. Then we'd have to figure out what changes would need to be made.  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment  
 

  
 

  
 
 
 
  
 

  
 
 
 
 

 
 This message was sent by Atlassian JIRA (v7.1.7#71011-sha1:2526d7c)  
 
 

 
   
 

  
 

  
 

   





-- 
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-issues+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


[JIRA] (JENKINS-41063) SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields

2017-01-13 Thread o.v.nenas...@gmail.com (JIRA)
Title: Message Title


 
 
 
 

 
 
 

 
   
 Oleg Nenashev created an issue  
 

  
 
 
 
 

 
 
  
 
 
 
 

 
 Jenkins /  JENKINS-41063  
 
 
  SSE GW Plugin: Serializable EventDispatcher contains non-serializable fields   
 

  
 
 
 
 

 
Issue Type: 
  Bug  
 
 
Assignee: 
 Tom FENNELLY  
 
 
Components: 
 sse-gateway-plugin  
 
 
Created: 
 2017/Jan/13 12:41 PM  
 
 
Priority: 
  Critical  
 
 
Reporter: 
 Oleg Nenashev  
 

  
 
 
 
 

 
 I was prototyping some code around Session management in Jenkins with standard Jetty's HashSessionManager. This session manager tries to serialize sessions and their attributes to the disk. In Jetty all session attributes are supposed to be serializable, especially if they declare the Serializable interface. In SSE Plugin EventDispather instances cannot be really serialized, because classes like PubSubBus and Authentication instances are not serializable in Jenkins core it causes failures in Session management.  
 

  
 
 
 
 

 
 
 

 
 
 Add Comment