You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/11/03 23:08:06 UTC

[GitHub] [arrow-datafusion] andygrove opened a new issue, #4103: The percentile argument for ApproxPercentileCont must be Float64, not Decimal128(2, 1)

andygrove opened a new issue, #4103:
URL: https://github.com/apache/arrow-datafusion/issues/4103

   **Describe the bug**
   I found this when testing https://github.com/apache/arrow-datafusion/pull/4102. `csv_query_approx_percentile_cont` fails with:
   
   ```
   The percentile argument for ApproxPercentileCont must be Float64, not Decimal128(2, 1)
   ```
   
   **To Reproduce**
   Run tests in https://github.com/apache/arrow-datafusion/pull/4102
   
   **Expected behavior**
   I assume we need to add type coercion to convert function args to the correct type
   
   **Additional context**
   


-- 
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-unsubscribe@arrow.apache.org.apache.org

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

[GitHub] [arrow-datafusion] comphead commented on issue #4103: The percentile argument for ApproxPercentileCont must be Float64, not Decimal128(2, 1)

Posted by GitBox <gi...@apache.org>.
comphead commented on issue #4103:
URL: https://github.com/apache/arrow-datafusion/issues/4103#issuecomment-1312589117

   Local test to reproduce
   ```
   #[tokio::test]
   async fn test_approx_percentile_cont() -> Result<()> {
       let ctx = SessionContext::new();
       register_aggregate_csv(&ctx).await?;
       let sql = "SELECT approx_percentile_cont(c2, cast(1.0 as decimal(10,2))) FROM aggregate_test_100";
       execute_to_batches(&ctx, sql).await;
   
       Ok(())
   }
   ```


-- 
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-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-datafusion] comphead commented on issue #4103: The percentile argument for ApproxPercentileCont must be Float64, not Decimal128(2, 1)

Posted by GitBox <gi...@apache.org>.
comphead commented on issue #4103:
URL: https://github.com/apache/arrow-datafusion/issues/4103#issuecomment-1310869687

   @andygrove please assign this to me


-- 
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-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-datafusion] alamb closed issue #4103: The percentile argument for ApproxPercentileCont must be Float64, not Decimal128(2, 1)

Posted by GitBox <gi...@apache.org>.
alamb closed issue #4103: The percentile argument for ApproxPercentileCont must be Float64, not Decimal128(2, 1)
URL: https://github.com/apache/arrow-datafusion/issues/4103


-- 
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-unsubscribe@arrow.apache.org

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