[GitHub] [hadoop-ozone] elek commented on pull request #1411: HDDS-4097. [DESIGN] S3/Ozone Filesystem inter-op

[GitBox]

elek commented on pull request #1411:
URL: https://github.com/apache/hadoop-ozone/pull/1411#issuecomment-691630554


   > > > Thank You @elek for the design document.
   > > > My understanding from this is the draft is as below. Let me know if I 
am missing something here.
   > > > <img alt="Screen Shot 2020-09-09 at 10 59 11 AM" width="998" 
src="https://user-images.githubusercontent.com/8586345/92635994-8856fe80-f28b-11ea-95bf-8864d48e488f.png";>
   > > 
   > > 
   > > Correct. But this is not a matrix anymore. You should turn on either 
first or second of the configs, but not both.
   > 
   > Not sure what is meant here, because we have 2 configs, now we can have 4 
combinations according to proposal 3 are valid, 4th one is not.
   
   Agree, but there are two ways to define this 3 options:
   
   1st approach:
   
   KEY1=true,KEY2=true --> option1
   KEY1=false,KEY2=false --> option2
   KEY1=true,KEY2=false --> option3
   KEY1=false,KEY2=true --> invalid 
   
   2nd approach:
   
   KEY1=true --> option 1
   KEY2=true --> option2
   else --> option3
   


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org