don4get commented on PR #43607:
URL: https://github.com/apache/arrow/pull/43607#issuecomment-2345983053

   > What is the error that you get when running it locally without the `noasm` 
flag? Did you expect there to be an error?
   > 
   > That particular flag controls whether or not optimized assembly code is 
utilized for decoding values, when that flag is set the library will fall back 
to pure go implementations of several decoding aspects such as unpacking 
booleans or packed integers. Without that flag, SIMD (vectorized) 
implementations of those functions are used instead on a per-architecture basis 
(falling back to pure go if we don't have optimized assembly for a given 
platform).
   
   
   ```
       file_reader_test.go:409: 
                Error Trace:    
/arrow/go/parquet/pqarrow/file_reader_test.go:409
                Error:          An error is expected but got nil.
   ```
   
   ```
        _, err = arrowRdr.ReadTable(ctx)
   
        assert.Error(t, err)
   ```
   
https://github.com/don4get/arrow/blob/589c3e4df00ca906c65830fa4e90ca182f424276/go/parquet/pqarrow/file_reader_test.go#L409
   
   While the default implementation fails to read the table (it returns an 
error), the `noasm` version does not fail (does not return an error).
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to