-
Notifications
You must be signed in to change notification settings - Fork 48
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
docs: add nginx access log analysis example #686
docs: add nginx access log analysis example #686
Conversation
f918e9d
to
185106d
Compare
Hi again @jovezhong, I've added another article to the examples. Hope you'll take a look 😉 . |
Nice! I will check it soon. ETA: end of this week, if not sooner. |
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.
Hi @ayewo , this is certainly in the right direction. This is my first batch of the comments. Look forwards to more examples of how to use Timeplus Proton to understand the blog visit pattern in the past, troubleshotting/tuning, and even understand what's going on right now.
def7de4
to
3b429ea
Compare
@jovezhong This is ready for review. |
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.
Great work, thanks for putting things together. I left some comments. One nice to have thing is to use Grafana proton datasource plugin to visualize the SQL result, or use metabase plugin. But optional for sure.
examples/nginx-access-logs-streaming/scripts/sql/01_create-tables.sql
Outdated
Show resolved
Hide resolved
examples/nginx-access-logs-streaming/scripts/sql/01_create-tables.sql
Outdated
Show resolved
Hide resolved
I struggled with getting a column in the access log to be parsed correctly as a SELECT parseDateTime('24/Apr/2024:15:49:59 +0000', '%d/%b/%Y:%H:%i:%s %z'); Works fine on ClickHouse: https://play.clickhouse.com/play?user=play#U0VMRUNUIHBhcnNlRGF0ZVRpbWUoJzI0L0Fwci8yMDI0OjE1OjQ5OjU5ICswMDAwJywgJyVkLyViLyVZOiVIOiVpOiVzICV6Jyk7Cgo= It seems Proton doesn't yet support the |
Hi @ayewo , regarding parsing '24/Apr/2024:15:49:59 +0000', can you create a new GitHub issue or discussion? We will implement it soon. |
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.
Thanks for the refinement for blog/images/code. Look good to me. I will merge it now.
27c6219
to
4fd1584
Compare
/tip 225 |
🎉🎈 @ayewo has been awarded $225! 🎈🎊 |
PR checklist:
proton: starts/ends
for new code in existing community code base ? N/A