https://pinot.apache.org/ logo
#general
Title
# general
z

Zsolt Takacs

06/23/2021, 3:47 PM
In the docs at https://docs.pinot.apache.org/users/tutorials/schema-evolution#update-the-schema Is this section still accurate? I can't find any usage of the property in the code.
Real-Time Pinot table: In case of real-time tables, make sure the "_pinot.server.instance.reload.consumingSegment_" config is set to true inside Server config. Without this, the current consuming segment(s) will not reflect the default null value for newly added columns.
m

Mayank

06/23/2021, 4:13 PM
You are right, seems like the default value was switched to true in this PR. https://github.com/apache/incubator-pinot/pull/7078
This was just merged a couple of days ago. The doc is still as per the prior official release.
cc @User
z

Zsolt Takacs

06/23/2021, 4:36 PM
I mean I grepped through the history and I couldn't find a reference actually reading the config value. I don't think it was ever actually wired. It was weird because we haven't set it and adding fields worked.
m

Mayank

06/23/2021, 4:38 PM
Are you saying it works with 0.7.1. release?
z

Zsolt Takacs

06/23/2021, 4:39 PM
yes
j

Jackie

06/23/2021, 4:41 PM
@User The next consuming segment will always get the updated schema. The feature is about whether the current consuming segment can add the new columns on the fly via reload
z

Zsolt Takacs

06/23/2021, 4:47 PM
I see, I've found that the usage is through a different constant