You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@seatunnel.apache.org by GitBox <gi...@apache.org> on 2022/10/12 01:50:55 UTC

[GitHub] [incubator-seatunnel] FWLamb opened a new issue, #3071: [Disscuss][api] Add New Data Type : BigInteger

FWLamb opened a new issue, #3071:
URL: https://github.com/apache/incubator-seatunnel/issues/3071

   ### Search before asking
   
   - [X] I had searched in the [feature](https://github.com/apache/incubator-seatunnel/issues?q=is%3Aissue+label%3A%22Feature%22) and found no similar feature requirement.
   
   
   ### Description
   
   Clickhouse needs to support Int128 and Int256. How to implement it? At present, adding BigInteger data type is a solution.
   
   ### Usage Scenario
   
   _No response_
   
   ### Related issues
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
   


-- 
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: commits-unsubscribe@seatunnel.apache.org.apache.org

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

[GitHub] [incubator-seatunnel] Hisoka-X commented on issue #3071: [Disscuss][api] Add New Data Type : BigInteger

Posted by GitBox <gi...@apache.org>.
Hisoka-X commented on issue #3071:
URL: https://github.com/apache/incubator-seatunnel/issues/3071#issuecomment-1275512283

   @ashulin PTAL.


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] ashulin commented on issue #3071: [Disscuss][api] Add New Data Type : BigInteger

Posted by GitBox <gi...@apache.org>.
ashulin commented on issue #3071:
URL: https://github.com/apache/incubator-seatunnel/issues/3071#issuecomment-1276236542

   I recommend using `decimal(38,0)`. 
   There is currently a BIGINT type, of course it only represents 8-byte data;
   This is also the current solution such as Impala, spark, and flink;


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] FWLamb commented on issue #3071: [Disscuss][api] Add New Data Type : BigInteger

Posted by GitBox <gi...@apache.org>.
FWLamb commented on issue #3071:
URL: https://github.com/apache/incubator-seatunnel/issues/3071#issuecomment-1278613150

   Problem solved.


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] FWLamb commented on issue #3071: [Disscuss][api] Add New Data Type : BigInteger

Posted by GitBox <gi...@apache.org>.
FWLamb commented on issue #3071:
URL: https://github.com/apache/incubator-seatunnel/issues/3071#issuecomment-1275484202

   If the Long data type is used, the _clickhouse client_ may overflow when the data exceeds the maximum value of Long The _clickhouse client_ uses BigInteger internally to process Int128 and Int256, or how we should distinguish them so that the _clickhouse client_ can use the correct data type.
   


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] FWLamb closed issue #3071: [Disscuss][api] Add New Data Type : BigInteger

Posted by GitBox <gi...@apache.org>.
FWLamb closed issue #3071: [Disscuss][api] Add New Data Type : BigInteger 
URL: https://github.com/apache/incubator-seatunnel/issues/3071


-- 
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: commits-unsubscribe@seatunnel.apache.org

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

[GitHub] [incubator-seatunnel] FWLamb commented on issue #3071: [Disscuss][api] Add New Data Type : BigInteger

Posted by GitBox <gi...@apache.org>.
FWLamb commented on issue #3071:
URL: https://github.com/apache/incubator-seatunnel/issues/3071#issuecomment-1276919733

   Good idea. I'll try it.


-- 
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: commits-unsubscribe@seatunnel.apache.org

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