Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Adds logs, fix for index creation date -1L, nullable checks #170

Merged
merged 3 commits into from
Apr 6, 2020

Conversation

dbbaughe
Copy link
Contributor

@dbbaughe dbbaughe commented Apr 3, 2020

Issue #, if available:

Description of changes:

  • The index creation_date defaults to -1L if there is none. I'm not sure if this ever actually happens, but adding logic/tests to treat it accordingly.
  • Adds tests around the evaluate methods
  • Adds logs before transition/rollover execution with the values when they evaluate to true.
  • Added kotlin compiler setting to not compile for nullable values not handled correctly w/ java interop. (Thanks Vinoo!)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@dbbaughe dbbaughe requested review from jinsoor-amzn and qreshi April 3, 2020 16:26
@dbbaughe dbbaughe changed the title Fixes Adds logs, fix for index creation date -1L, nullable checks Apr 3, 2020
@dbbaughe dbbaughe merged commit f23b5c2 into opendistro-for-elasticsearch:master Apr 6, 2020
dbbaughe added a commit to dbbaughe/index-management that referenced this pull request Apr 6, 2020
…ro-for-elasticsearch#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log
@dbbaughe dbbaughe mentioned this pull request Apr 6, 2020
dbbaughe added a commit to dbbaughe/index-management that referenced this pull request Apr 6, 2020
…ro-for-elasticsearch#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log
@dbbaughe dbbaughe mentioned this pull request Apr 6, 2020
dbbaughe added a commit that referenced this pull request Apr 6, 2020
* Adds logs, fix for index creation date -1L, nullable checks (#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log

* Delete and close failing during snapshot in progress (#172)

* Fixes AttemptDeleteStep failing from a SnapshotInProgressException

* Fixes close action failing on snapshot in progress exception, fixes imports

* Fixes styling
dbbaughe added a commit that referenced this pull request Apr 6, 2020
* Adds logs, fix for index creation date -1L, nullable checks (#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log

* Delete and close failing during snapshot in progress (#172)

* Fixes AttemptDeleteStep failing from a SnapshotInProgressException

* Fixes close action failing on snapshot in progress exception, fixes imports

* Fixes styling
dbbaughe added a commit to dbbaughe/index-management that referenced this pull request Apr 6, 2020
…ro-for-elasticsearch#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log
dbbaughe added a commit to dbbaughe/index-management that referenced this pull request Apr 6, 2020
…ro-for-elasticsearch#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log
dbbaughe added a commit to dbbaughe/index-management that referenced this pull request Apr 6, 2020
…ro-for-elasticsearch#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log
dbbaughe added a commit to dbbaughe/index-management that referenced this pull request Apr 6, 2020
…ro-for-elasticsearch#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log
dbbaughe added a commit to dbbaughe/index-management that referenced this pull request Apr 6, 2020
…ro-for-elasticsearch#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log
dbbaughe added a commit that referenced this pull request Apr 7, 2020
* Adds logs, fix for index creation date -1L, nullable checks (#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log

* Delete and close failing during snapshot in progress (#172)

* Fixes AttemptDeleteStep failing from a SnapshotInProgressException

* Fixes close action failing on snapshot in progress exception, fixes imports

* Fixes styling

* Fixes CloseIndexResponse init

* Fixes test to use Ack response

* Fixes ack value
dbbaughe added a commit that referenced this pull request Apr 7, 2020
* Adds logs, fix for index creation date -1L, nullable checks (#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log

* Delete and close failing during snapshot in progress (#172)

* Fixes AttemptDeleteStep failing from a SnapshotInProgressException

* Fixes close action failing on snapshot in progress exception, fixes imports

* Fixes styling

* Fixes CloseIndexResponse init

* Fixes missed merge conflict

* Fixes tests
dbbaughe added a commit that referenced this pull request Apr 7, 2020
* Adds logs, fix for index creation date -1L, nullable checks (#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log

* Delete and close failing during snapshot in progress (#172)

* Fixes AttemptDeleteStep failing from a SnapshotInProgressException

* Fixes close action failing on snapshot in progress exception, fixes imports

* Fixes styling

* Fixes CloseIndexResponse init

* Fixes missed merge conflict

* Fixes tests
dbbaughe added a commit that referenced this pull request Apr 7, 2020
* Adds logs, fix for index creation date -1L, nullable checks (#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log

* Delete and close failing during snapshot in progress (#172)

* Fixes AttemptDeleteStep failing from a SnapshotInProgressException

* Fixes close action failing on snapshot in progress exception, fixes imports

* Fixes styling

* Fix CloseIndexResponse init
dbbaughe added a commit that referenced this pull request Apr 7, 2020
* Adds logs, fix for index creation date -1L, nullable checks (#170)

* Index creation_date of -1L should evaluate to false, adds extra logs

* Adds kotlin compiler check and fixes nullable values

* Adds log

* Delete and close failing during snapshot in progress (#172)

* Fixes AttemptDeleteStep failing from a SnapshotInProgressException

* Fixes close action failing on snapshot in progress exception, fixes imports

* Fixes styling
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants