Skip to content
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

PropertiesColumnWriter: ability to exclude specified properties #8

Open
SeppPenner opened this issue May 12, 2019 · 3 comments
Open
Assignees
Labels
enhancement New feature or request

Comments

@SeppPenner
Copy link
Contributor

See b00ted/serilog-sinks-postgresql#21 and b00ted/serilog-sinks-postgresql#22

@SeppPenner
Copy link
Contributor Author

I still do not see the benefit: b00ted/serilog-sinks-postgresql#21 (comment)

@SeppPenner
Copy link
Contributor Author

Will be closed until someone responds in b00ted/serilog-sinks-postgresql#21

@Gaz83
Copy link

Gaz83 commented Mar 2, 2023

Just stumbled upon this, this would be a great feature.
We are currently having a look at switching over to Linux and using PostgreSQL. I have been using Serilog with the MS SQL sink and this has 2 nice features that I use.

1)If there is a column for a property then don't also store it in the properties column. This is controlled by a settings in the config so, which is off by default.

Example, If there is a property, say "RequestContent", and you create a column for this property then the property is removed from the properties column automatically, no need to write it twice.

Before
Column Properties: "{ "Properties": [ "RequestContent": "blah blah blah", "ActionId": "SomeID" ]}"
Column RequestContent: "blah blah blah"

After with auto property removed
Column Properties: { "Properties": [ "ActionId": "SomeID" ]}
Column RequestContent: "blah blah blah"

  1. Being able to specify which standard columns you want/don't want. I use this feature a lot and in one particular Table I actually don't use the LogEvent or Properties columns as I have created columns for the parts I care about. This make it more efficient for querying.

Hope this all makes sense.

@SeppPenner SeppPenner reopened this Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants