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

Bad assertion in serializePartitionKey when shortKeys are enabled #17

Closed
tolbertam opened this issue Dec 18, 2015 · 0 comments
Closed
Milestone

Comments

@tolbertam
Copy link
Owner

When running with assertions enabled and no schema is provided via '-c' toJson will fail while serializing a partition key because it expects the number of key components to match the number of partition columns in the metadata (which are empty):

Exception in thread "main" java.lang.AssertionError
    at com.csforge.sstable.JsonTransformer.serializePartitionKey(JsonTransformer.java:83)
    at com.csforge.sstable.JsonTransformer.serializePartition(JsonTransformer.java:149)
    at java.util.stream.ForEachOps$ForEachOp$OfRef.accept(ForEachOps.java:184)
    at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
    at java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:175)
    at java.util.Iterator.forEachRemaining(Iterator.java:116)
    at java.util.Spliterators$IteratorSpliterator.forEachRemaining(Spliterators.java:1801)
    at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
    at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
    at java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:151)
    at java.util.stream.ForEachOps$ForEachOp$OfRef.evaluateSequential(ForEachOps.java:174)
    at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
    at java.util.stream.ReferencePipeline.forEach(ReferencePipeline.java:418)
    at com.csforge.sstable.JsonTransformer.toJson(JsonTransformer.java:58)
    at com.csforge.sstable.SSTable2Json.main(SSTable2Json.java:109)
    at com.csforge.sstable.Driver.main(Driver.java:17)
@tolbertam tolbertam added this to the 3.0.0-alpha2 milestone Dec 18, 2015
tolbertam added a commit that referenced this issue Dec 18, 2015
[#17] Relax component == key count assertion if using shortKeys.
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