The timestamp field is marked as persisted but it is not

[cneth]

According to the help text, the 'timestamp' field is one of the items persisted. However, it does not actually work. It looks like that despite the comment in configuration.go about making sure to keep the CopyConfigRelevantSettingsTo contents in sync with the array of persisted flags, that didn't happen.

I believe the following patch solves the issues:

diff --git a/configuration.go b/configuration.go
index 573d8be..951672c 100644
--- a/configuration.go
+++ b/configuration.go
@@ -76,6 +76,7 @@ func (c *Configuration) CopyConfigRelevantSettingsTo(dest *Configuration) {
        dest.SearchTarget.Url = c.SearchTarget.Url
        dest.SearchTarget.IndexPattern = c.SearchTarget.IndexPattern
        dest.QueryDefinition.Format = c.QueryDefinition.Format
+       dest.QueryDefinition.TimestampField = c.QueryDefinition.TimestampField
        dest.QueryDefinition.Terms = make([]string, len(c.QueryDefinition.Terms))
        copy(dest.QueryDefinition.Terms, c.QueryDefinition.Terms)
        dest.User = c.User
@@ -84,7 +85,6 @@ func (c *Configuration) CopyConfigRelevantSettingsTo(dest *Configuration) {
 
 func (c *Configuration) CopyNonConfigRelevantSettingsTo(dest *Configuration) {
        //copy non-config relevant settings
-       dest.QueryDefinition.TimestampField = c.QueryDefinition.TimestampField
        dest.QueryDefinition.AfterDateTime = c.QueryDefinition.AfterDateTime
        dest.QueryDefinition.BeforeDateTime = c.QueryDefinition.BeforeDateTime
        dest.ListOnly = c.ListOnly

[knes1]

Hi! Thank you for noticing this and for sending over the patch. It looks good to me. I'll be merging this in over the weekend.