slfan1989 commented on PR #4435:
URL: https://github.com/apache/hadoop/pull/4435#issuecomment-1499906514
> I had the same problem. hadoop 3.3.5 was released and had the same problem
We will continue to follow up this pr. @LennonChin Can we rebuild this pr?
--
This is an automated
slfan1989 commented on PR #4435:
URL: https://github.com/apache/hadoop/pull/4435#issuecomment-1438320061
@LennonChin Is this junit test error related to this pr?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
slfan1989 commented on PR #4435:
URL: https://github.com/apache/hadoop/pull/4435#issuecomment-1414736328
@LennonChin Thank you for your contribution! Is the unit test error caused
by this pr?
--
This is an automated message from the Apache Git Service.
To respond to the message, please
slfan1989 commented on PR #4435:
URL: https://github.com/apache/hadoop/pull/4435#issuecomment-1362744741
@LennonChin can we rebase again?
--
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
slfan1989 commented on PR #4435:
URL: https://github.com/apache/hadoop/pull/4435#issuecomment-1158503105
@LennonChin LGTM. Thanks for your contribution, it looks like it's working
well, let's wait for other buddies' suggestions.
--
This is an automated message from the Apache Git
slfan1989 commented on PR #4435:
URL: https://github.com/apache/hadoop/pull/4435#issuecomment-1158379831
> @slfan1989 @dineshchitlangia @brumi1024 @9uapaw @ashutoshcipher Could you
please to help review the code?
I understand your change, and it seems feasible, can we give the data
slfan1989 commented on PR #4435:
URL: https://github.com/apache/hadoop/pull/4435#issuecomment-1158378152
> @slfan1989 @dineshchitlangia @brumi1024 @9uapaw @ashutoshcipher Could you
please to help review the code?
I have understood your changes, but can the token also add some user
slfan1989 commented on PR #4435:
URL: https://github.com/apache/hadoop/pull/4435#issuecomment-1153743066
@LennonChin I ask a question, Why do you need to add the synchronized
keyword?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to