You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2021/05/04 21:01:08 UTC

[GitHub] [accumulo] EdColeman opened a new issue #2078: Variables in org.apache.accumulo.fate.FATE class can be final.

EdColeman opened a new issue #2078:
URL: https://github.com/apache/accumulo/issues/2078


   The fields store, environment and executor can be marked final in [FATE](https://github.com/apache/accumulo/blob/552fd33407f9346fba45525f3abb36864de2ea09/core/src/main/java/org/apache/accumulo/fate/Fate.java#L51)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [accumulo] Akshit6828 commented on issue #2078: Variables in org.apache.accumulo.fate.FATE class can be final.

Posted by GitBox <gi...@apache.org>.
Akshit6828 commented on issue #2078:
URL: https://github.com/apache/accumulo/issues/2078#issuecomment-832250171


   Hi @EdColeman  can i work on this issue?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [accumulo] Akshit6828 commented on issue #2078: Variables in org.apache.accumulo.fate.FATE class can be final.

Posted by GitBox <gi...@apache.org>.
Akshit6828 commented on issue #2078:
URL: https://github.com/apache/accumulo/issues/2078#issuecomment-832275624


   No problem @EdColeman  sir. I'll fix this up and submit a PR.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [accumulo] Akshit6828 commented on issue #2078: Variables in org.apache.accumulo.fate.FATE class can be final.

Posted by GitBox <gi...@apache.org>.
Akshit6828 commented on issue #2078:
URL: https://github.com/apache/accumulo/issues/2078#issuecomment-832275194


   Sure @EdColeman  . Yeah, I saw the checks. It's generating an error as: cannot assign a value to a final variable executor.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [accumulo] EdColeman commented on issue #2078: Variables in org.apache.accumulo.fate.FATE class can be final.

Posted by GitBox <gi...@apache.org>.
EdColeman commented on issue #2078:
URL: https://github.com/apache/accumulo/issues/2078#issuecomment-832274235


   My mistake - executor cannot be final - the other filed that can be final is keepRunning - would you correct and push the changes to your PR - sorry for the confusion.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [accumulo] EdColeman closed issue #2078: Variables in org.apache.accumulo.fate.FATE class can be final.

Posted by GitBox <gi...@apache.org>.
EdColeman closed issue #2078:
URL: https://github.com/apache/accumulo/issues/2078


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org