Discussion:
Drill Log file plugin not working in Drill 1.14
Divya Gehlot
2018-10-04 05:09:40 UTC
Permalink
Hi,
I am using Drill 1.14 and followed these instructions
<https://drill.apache.org/docs/logfile-plugin/>to read log files of any
format in drill.

When I insert the log plugin in the storage plugin , it is complaining that
its is not valid JSON , though I already verified it through JSON
validator and it seems to be correct .
Pasted the storage plugin here <https://pastebin.com/ATWxHFqQ>

Thanks,
Divya
Charles Givre
2018-10-04 14:52:56 UTC
Permalink
Hi Divya,
Good catch. These docs are for an earlier version of this plugin that didn’t make it into Drill. Here are correct docs:

https://github.com/cgivre/drill/blob/24556d857cbbe7aa2baa1fc6cbd85fb614b5d975/exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md <https://github.com/cgivre/drill/blob/24556d857cbbe7aa2baa1fc6cbd85fb614b5d975/exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md>

You’ll find that the newer version is A LOT easier to use than that!


@Bridget, could you please take a look at this and update the docs? Thanks!
Post by Divya Gehlot
Hi,
I am using Drill 1.14 and followed these instructions
<https://drill.apache.org/docs/logfile-plugin/>to read log files of any
format in drill.
When I insert the log plugin in the storage plugin , it is complaining that
its is not valid JSON , though I already verified it through JSON
validator and it seems to be correct .
Pasted the storage plugin here <https://pastebin.com/ATWxHFqQ>
Thanks,
Divya
Divya Gehlot
2018-10-08 06:45:58 UTC
Permalink
Thank you so much Paul !
It worked perfectly :)
Can I be help in updating the docs ? Let me know .
I am not sure whether I have the permission to do so .

Thanks,
Divya
Post by Charles Givre
Hi Divya,
Good catch. These docs are for an earlier version of this plugin that
https://github.com/cgivre/drill/blob/24556d857cbbe7aa2baa1fc6cbd85fb614b5d975/exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md
<
https://github.com/cgivre/drill/blob/24556d857cbbe7aa2baa1fc6cbd85fb614b5d975/exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md
You’ll find that the newer version is A LOT easier to use than that!
@Bridget, could you please take a look at this and update the docs?
Thanks!
Post by Divya Gehlot
Hi,
I am using Drill 1.14 and followed these instructions
<https://drill.apache.org/docs/logfile-plugin/>to read log files of any
format in drill.
When I insert the log plugin in the storage plugin , it is complaining
that
Post by Divya Gehlot
its is not valid JSON , though I already verified it through JSON
validator and it seems to be correct .
Pasted the storage plugin here <https://pastebin.com/ATWxHFqQ>
Thanks,
Divya
Arina Yelchiyeva
2018-10-08 07:40:26 UTC
Permalink
Hi Divya,

You can create PR for docs update. Charles and Bridget can review it.
Example of the PR - https://github.com/apache/drill/pull/1479 <https://github.com/apache/drill/pull/1479>

Kind regards,
Arina
Post by Divya Gehlot
Thank you so much Paul !
It worked perfectly :)
Can I be help in updating the docs ? Let me know .
I am not sure whether I have the permission to do so .
Thanks,
Divya
Post by Charles Givre
Hi Divya,
Good catch. These docs are for an earlier version of this plugin that
https://github.com/cgivre/drill/blob/24556d857cbbe7aa2baa1fc6cbd85fb614b5d975/exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md
<
https://github.com/cgivre/drill/blob/24556d857cbbe7aa2baa1fc6cbd85fb614b5d975/exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md
You’ll find that the newer version is A LOT easier to use than that!
@Bridget, could you please take a look at this and update the docs?
Thanks!
Post by Divya Gehlot
Hi,
I am using Drill 1.14 and followed these instructions
<https://drill.apache.org/docs/logfile-plugin/>to read log files of any
format in drill.
When I insert the log plugin in the storage plugin , it is complaining
that
Post by Divya Gehlot
its is not valid JSON , though I already verified it through JSON
validator and it seems to be correct .
Pasted the storage plugin here <https://pastebin.com/ATWxHFqQ>
Thanks,
Divya
Divya Gehlot
2018-10-11 07:16:31 UTC
Permalink
Hi Arina,

Just to clarify If I create the pull request it will automatically gets
publish to https://drill.apache.org/docs/logfile-plugin/ , there is no
need update the docs?

Thanks,
Divya
Post by Charles Givre
Hi Divya,
You can create PR for docs update. Charles and Bridget can review it.
Example of the PR - https://github.com/apache/drill/pull/1479 <
https://github.com/apache/drill/pull/1479>
Kind regards,
Arina
Post by Divya Gehlot
Thank you so much Paul !
It worked perfectly :)
Can I be help in updating the docs ? Let me know .
I am not sure whether I have the permission to do so .
Thanks,
Divya
Post by Charles Givre
Hi Divya,
Good catch. These docs are for an earlier version of this plugin that
https://github.com/cgivre/drill/blob/24556d857cbbe7aa2baa1fc6cbd85fb614b5d975/exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md
Post by Divya Gehlot
Post by Charles Givre
<
https://github.com/cgivre/drill/blob/24556d857cbbe7aa2baa1fc6cbd85fb614b5d975/exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md
Post by Divya Gehlot
Post by Charles Givre
You’ll find that the newer version is A LOT easier to use than that!
@Bridget, could you please take a look at this and update the docs?
Thanks!
Post by Divya Gehlot
Hi,
I am using Drill 1.14 and followed these instructions
<https://drill.apache.org/docs/logfile-plugin/>to read log files of
any
Post by Divya Gehlot
Post by Charles Givre
Post by Divya Gehlot
format in drill.
When I insert the log plugin in the storage plugin , it is complaining
that
Post by Divya Gehlot
its is not valid JSON , though I already verified it through JSON
validator and it seems to be correct .
Pasted the storage plugin here <https://pastebin.com/ATWxHFqQ>
Thanks,
Divya
Arina Yelchiyeva
2018-10-11 08:59:02 UTC
Permalink
All PRs first undergo code review, even doc updates.
Then PRs are merged by the Apache committer.

Kind regards,
Arina
Post by Divya Gehlot
Hi Arina,
Just to clarify If I create the pull request it will automatically gets
publish to https://drill.apache.org/docs/logfile-plugin/ , there is no
need update the docs?
Thanks,
Divya
Post by Charles Givre
Hi Divya,
You can create PR for docs update. Charles and Bridget can review it.
Example of the PR - https://github.com/apache/drill/pull/1479 <
https://github.com/apache/drill/pull/1479>
Kind regards,
Arina
Post by Divya Gehlot
Thank you so much Paul !
It worked perfectly :)
Can I be help in updating the docs ? Let me know .
I am not sure whether I have the permission to do so .
Thanks,
Divya
Post by Charles Givre
Hi Divya,
Good catch. These docs are for an earlier version of this plugin that
https://github.com/cgivre/drill/blob/24556d857cbbe7aa2baa1fc6cbd85fb614b5d975/exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md
Post by Charles Givre
Post by Divya Gehlot
Post by Charles Givre
<
https://github.com/cgivre/drill/blob/24556d857cbbe7aa2baa1fc6cbd85fb614b5d975/exec/java-exec/src/main/java/org/apache/drill/exec/store/log/README.md
Post by Charles Givre
Post by Divya Gehlot
Post by Charles Givre
You’ll find that the newer version is A LOT easier to use than that!
@Bridget, could you please take a look at this and update the docs?
Thanks!
Post by Divya Gehlot
Hi,
I am using Drill 1.14 and followed these instructions
<https://drill.apache.org/docs/logfile-plugin/>to read log files of
any
Post by Divya Gehlot
Post by Charles Givre
Post by Divya Gehlot
format in drill.
When I insert the log plugin in the storage plugin , it is
complaining
Post by Charles Givre
Post by Divya Gehlot
Post by Charles Givre
that
Post by Divya Gehlot
its is not valid JSON , though I already verified it through JSON
validator and it seems to be correct .
Pasted the storage plugin here <https://pastebin.com/ATWxHFqQ>
Thanks,
Divya
Loading...