milleruntime commented on pull request #1799: URL: https://github.com/apache/accumulo/pull/1799#issuecomment-732344630
> I'm a little confused by these changes. Are you trying to change the metadata schema to put all the locations in the same column family? I ask because this seems to ensure that the "location" column family is fetched when we're loading the "last" location... and that doesn't make sense to me, because they are in separate column families today. Same with suspend. This change is motivated by discussion on #1317 . I added this method to Ample to get the tablet state. The class where this is done is `TabletLocationState` and requires multiple column families (current, future, last and suspend), all having to do with tablet location. To make it easier to do this in Ample, I made the `ColumnType.LOCATION` fetch all four because it seemed confusing to fetch the famillies individually when they are all related. ---------------------------------------------------------------- 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: [email protected]
