docs: add nginx access log analysis example#686
docs: add nginx access log analysis example#686jovezhong merged 36 commits intotimeplus-io:developfrom
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. |
jovezhong
left a comment
There was a problem hiding this comment.
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. |
jovezhong
left a comment
There was a problem hiding this comment.
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. |
jovezhong
left a comment
There was a problem hiding this comment.
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/endsfor new code in existing community code base ? N/A