claudevdm commented on PR #34699:
URL: https://github.com/apache/beam/pull/34699#issuecomment-2828564354

   There are a bunch of failing tests e.g.
   ```
   self = <apache_beam.internal.cloudpickle.cloudpickle.PurePythonPickler 
object at 0x7a8974470940>
   obj = b''
   
       def memoize(self, obj):
           """Store an object in the memo."""
       
           # The Pickler memo is a dictionary mapping object ids to 2-tuples
           # that contain the Unpickler memo key and the object being memoized.
           # The memo key is written to the pickle and will become
           # the key in the Unpickler's memo.  The object is stored in the
           # Pickler memo so that transient objects are kept alive during
           # pickling.
       
           # The use of the Unpickler memo length as the memo key is just a
           # convention.  The only requirement is that the memo values be 
unique.
           # But there appears no advantage to any other scheme, and this
           # scheme allows the Unpickler memo to be implemented as a plain (but
           # growable) array, indexed by memo key.
           if self.fast:
               return
   >       assert id(obj) not in self.memo
   E       AssertionError
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to