You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@knox.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2022/05/17 13:13:00 UTC

[jira] [Work logged] (KNOX-2346) Remove unused maxRetryAttempts and retrySleep

     [ https://issues.apache.org/jira/browse/KNOX-2346?focusedWorklogId=771335&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-771335 ]

ASF GitHub Bot logged work on KNOX-2346:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 17/May/22 13:12
            Start Date: 17/May/22 13:12
    Worklog Time Spent: 10m 
      Work Description: smolnar82 opened a new pull request, #577:
URL: https://github.com/apache/knox/pull/577

   ## What changes were proposed in this pull request?
   
   Removed the previously eliminated `maxRetryAttempts` and `retrySleep` HA configs from a test case which were accidentally re-introduced.
   
   ## How was this patch tested?
   
   Unit testing.
   




Issue Time Tracking
-------------------

    Worklog Id:     (was: 771335)
    Time Spent: 20m  (was: 10m)

> Remove unused maxRetryAttempts and retrySleep
> ---------------------------------------------
>
>                 Key: KNOX-2346
>                 URL: https://issues.apache.org/jira/browse/KNOX-2346
>             Project: Apache Knox
>          Issue Type: Task
>            Reporter: Kevin Risden
>            Assignee: Sandor Molnar
>            Priority: Major
>             Fix For: 1.5.0
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Currently maxRetryAttempts and retrySleep are wired throughout the code, but don't actually hook up to anything. The handling should be removed to make it clear that these aren't used anywhere.
> There is no backwards compatibility issue here since the parsing is lazy - its not a required field so if a user is specifying it will be silently ignore. Just like it is silently ignored today.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)