Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Need support for bigint/int unsigned type #3

Closed
vikas22 opened this issue Dec 13, 2018 · 4 comments
Closed

Need support for bigint/int unsigned type #3

vikas22 opened this issue Dec 13, 2018 · 4 comments

Comments

@vikas22
Copy link
Contributor

vikas22 commented Dec 13, 2018

Ex: schema

| Field                    | Type                         | Null   | Key  | Default  |
| amount               | bigint(20) unsigned | NO   | MUL | NULL    |

Library throws

Exception in thread "main" java.lang.IllegalArgumentException: unknown data type: bigint unsigned
	at com.at.avro.config.AvroConfig.lambda$new$2(AvroConfig.java:30)
	at com.at.avro.AvroTypeUtil.getPrimitiveType(AvroTypeUtil.java:97)
	at com.at.avro.AvroTypeUtil.getAvroType(AvroTypeUtil.java:43)
	at com.at.avro.AvroField.<init>(AvroField.java:25)
	at com.at.avro.AvroSchema.<init>(AvroSchema.java:26)
	at com.at.avro.DbSchemaExtractor.get(DbSchemaExtractor.java:93)
	at com.at.avro.DbSchemaExtractor.getForTables(DbSchemaExtractor.java:57)
	at scala.Function0.apply$mcV$sp(Function0.scala:34)
	at scala.Function0.apply$mcV$sp$(Function0.scala:34)
	at scala.runtime.AbstractFunction0.apply$mcV$sp(AbstractFunction0.scala:12)
	at scala.App.$anonfun$main$1$adapted(App.scala:76)
	at scala.collection.immutable.List.foreach(List.scala:388)
	at scala.App.main(App.scala:76)
	at scala.App.main$(App.scala:74)
@vikas22
Copy link
Contributor Author

vikas22 commented Dec 13, 2018

@artur-tamazian @atfire #4 please check this

@vikas22 vikas22 closed this as completed Dec 13, 2018
@artur-tamazian
Copy link
Owner

I released a new version with your fix. Thanks for the contribution!

@vikas22 vikas22 reopened this Dec 20, 2018
@vikas22
Copy link
Contributor Author

vikas22 commented Dec 20, 2018

a0e49dc @artur-tamazian needed support for smallint unsigned and mediumint please approve

@artur-tamazian
Copy link
Owner

@vikas22 thank you for another contribution! Version 1.0.2 released

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants