QingdongZeng3 commented on pull request #13690:
URL: https://github.com/apache/flink/pull/13690#issuecomment-715322368


   > > > Have you tested this change with multiple kerberos-enabled NameNodes?
   > 
   > > yes ,It has works in my production environment for a long time.
   > 
   > Okay, that's great to hear! If extending the e2e tests turns out to be 
extremely difficult, I would also be fine with merging this change without 
tests, given that this is a non critical add on feature that is difficult to 
test.
   > But first it would be nice if you could look into the e2e tests. In my 
opinion it should be possible. Sneaking in the right 2nd namenode config could 
be a bit difficult, but there's probably a HADOOP_CONF_DIR environment variable 
(or HADOOP_CLASSPATH) variable that you need to temporary overwrite for the 2nd 
NameNode)
   
   Hi Robert,
   Through team members, I learned that we use our own independent test system 
and don't have an environment for E2E tests using docker, it seems difficult to 
implement this test. As you say, this is a non critical add on feature  , and  
it wouldn't affect any other functions, so I  expect to merge this change 
without e2e test,Thanks!


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to