Need to implement shared import

GoogleCodeExporter commented 8 years ago

*What steps will reproduce the problem?*
1. Follow the directions on 
http://code.google.com/p/walkaround/wiki/RunningTheCode?repo=wiki to create a 
new application on appspot.com
2. Go to https://mywalkaround.appspot.com/import
3. Find waves to import.
4. Import a wave.
5. A task appears: "Shared import of wavelet googlewave.com!blabla

*What is the expected output?*
1. I expect the task to be gone when I refresh the page after a minute or so.

*What do you see instead?*
1. The task never disappears.

*What browser and browser version are you using?  On what operating system?*
1. N/a.

*What URL does your browser show when the problem occurs?  Did you compile 
walkaround on your machine, or are you using a public instance?*
1. https://mywalkaround.appspot.com/import
2. http://localhost:8080/import

*Please provide any additional information below.*
Behavior is consistent between appspot and localhost.
On localhost I can watch the command line. At one point it says:
  [java] timestamp PM co.google.walkaround.wave.server.googleimport.conversion.HistorySynthesizer pckValidParticipantId
  [java] INFO: Encountered invalid participent address: <nobody>
  [java] timestamp PM co.google.walkaround.wave.server.googleimport.conversion.HistorySynthesizer pckValidParticipantId
  [java] INFO: Encountered invalid participent address: <nobody>
  [java] timestamp PM com.google.walkaround.wave.server.servlet.ServerExceptionFilter sendError
  [java] SEVERE: RuntimeException; sending 500: Internal server error
  [java] java.lang.RuntimeException: not implemented

The specific wave has the following participants:
- Me
- someone@googlewave.com
- groupname@googlegroups.com
- someone1@googlewave.com
- someone2@googlewave.com
- someone3@googlewave.com
- emoticonbot@appspot.com

Though the command line lists two <body> lines and the wave has two 
non-googlewave.com participants, also waves that have only googlewave.com 
participants fail to import with the same error.

Guillaume Hanique

Original issue reported on code.google.com by guillaum...@gmail.com on 13 Jan 2012 at 1:15

GoogleCodeExporter commented 8 years ago

Thanks for the report.  Shared import is not yet implemented and will fail for 
any wave.  I'll be adding this feature soon.  Sorry for the confusing user 
interface, I guess I should have removed the button.  You can use private 
import for now.

The "Encountered invalid participant address" message is only an INFO message 
since the code knows how to deal with that situation; the actual error is the 
SEVERE message.  There should be a stack trace immediately following the 
section of the logs that you copied; in general, please include stack traces in 
bug reports, because they usually point to the code where the problem occurred. 
 (You should be able to use copy and paste rather than typing log messages into 
bugs manually.)

Original comment by oh...@google.com on 13 Jan 2012 at 7:06

Changed title: Need to implement shared import
Changed state: Accepted
Added labels: Component-Import

GoogleCodeExporter commented 8 years ago

Fixed in http://code.google.com/p/walkaround/source/detail?r=18fb56a50e8d .

Original comment by oh...@google.com on 24 Jan 2012 at 4:03

Changed state: Fixed

byMan / walkaround

Need to implement shared import #41