-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Does Pinot hybrid table allowed to be created with different schema #11549
Comments
No. I think we didn't enforce that before, but no longer allow such configuration going forward |
thanks @Jackie-Jiang for clarification. |
If this is not enforced already can we start enforcing it? @deemoliu you were able to create the hybrid table with different schemas with 0.11? |
there is an attached example in the description of this issue. |
The question was whether we hit this issue with 0.11 or a later version. Anyways looks like there's a PR already for this #11591 |
FYI, #11591 will enforce schema name to be the same as raw table config for each table, which also indices that hybrid table must use the same schema (both offline and real-time side have same raw table name). Why are you still running on 0.11? Are you planning to directly upgrade to 1.0? That is violating the upgrade policy of not jumping release versions |
@Jackie-Jiang : we do upgrades incrementally (1 minor version at a time). Some clusters are using 0.11, some are running 0.12. We have quite a large deployment and it's not easy to keep rolling out new versions. |
Context
We have created a hybrid table succesfully with different schema in realtime and offline part.
However we also find code, which infers realtime and offline table in one hybrid table should have one schema.
https://github.com/apache/pinot/blob/master/pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/TableConfigsRestletResource.java#L451-L464 cc: @ankitsultana
Does Pinot hybrid table allowed to be created with different schema?
The text was updated successfully, but these errors were encountered: