Add authorization check options to iceberg doc#10670
Conversation
There was a problem hiding this comment.
Is the 'iceberg.security' property in the catalog properties file by default or does the user need to add it and set the values?
If they have to add it to the file and choose a value, I suggest reframing the sentence to something like:
To enable authorization checks, add the 'iceberg.security' property to the connector catalog properties file.
There was a problem hiding this comment.
I don't disagree, but we've used the wording "set/by setting the example.property property in the catalog properties file" just about everywhere else in the connector docs. This property is no different, I'd leave it as-is.
|
@kokosing can we get your input here? |
|
@kokosing Can you please merge this PR if all looks good? Thanks! |
jhlodin
force-pushed
the
jl/iceberg-security
branch
2 times, most recently
from
7b81518 to
de79c03
Compare
Sure. Please ping me once CI gets a green light. |
jhlodin
force-pushed
the
jl/iceberg-security
branch
from
de79c03 to
cfafa02
Compare
jhlodin
force-pushed
the
jl/iceberg-security
branch
from
cfafa02 to
60ae126
Compare
4 participants
Issue: #10548