-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Fix protobuf use case document #11545
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks @bananaaggle
@bananaaggle Do you think you could take a stab at fixing the web-console to use the inputFormat instead of the parser in a future change? I don't know a lot about front-end coding but that would be awesome for people to be able to use the UI instead of having to hand craft the ingestion spec. Thanks again for all your contributions! |
Actually, all data formats in core extension have been adapted to |
I misunderstood when I wrote my last comment - I thought the web console was generating a deprecated parse spec when users selected protobuf, avro, etc. I agree with you that since the |
Now we have protobuf inputformat, and protobuf parser will be deprecated. So we should change document to follow it. Actually, because of 10647, supervisor properties with parser not work in 'Load data' tab in frontend, as described in 11537.
This PR has: