Uploaded image for project: 'Apache Arrow'
  1. Apache Arrow
  2. ARROW-2305

[Python] Cython 0.25.2 compilation failure

    Details

      Description

      Observed on master branch

      Error compiling Cython file:
      ------------------------------------------------------------
      ...
              if hasattr(self, 'as_py'):
                  return repr(self.as_py())
              else:
                  return super(Scalar, self).__repr__()
      
          def __eq__(self, other):
         ^
      ------------------------------------------------------------
      
      /home/wesm/code/arrow/python/pyarrow/scalar.pxi:67:4: Special method __eq__ must be implemented via __richcmp__
      
      Error compiling Cython file:
      ------------------------------------------------------------
      ...
              Return true if the tensors contains exactly equal data
              """
              self._validate()
              return self.tp.Equals(deref(other.tp))
      
          def __eq__(self, other):
         ^
      ------------------------------------------------------------
      
      /home/wesm/code/arrow/python/pyarrow/array.pxi:571:4: Special method __eq__ must be implemented via __richcmp__
      
      Error compiling Cython file:
      ------------------------------------------------------------
      ...
              cdef c_bool result = False
              with nogil:
                  result = self.buffer.get().Equals(deref(other.buffer.get()))
              return result
      
          def __eq__(self, other):
         ^
      ------------------------------------------------------------
      
      /home/wesm/code/arrow/python/pyarrow/io.pxi:675:4: Special method __eq__ must be implemented via __richcmp__
      

      Upgrading Cython made this go away. We should probably use _richcmp_ though

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                pitrou Antoine Pitrou
                Reporter:
                wesmckinn Wes McKinney
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: