ctubbsii commented on issue #4065: URL: https://github.com/apache/accumulo/issues/4065#issuecomment-1854892110
Okay, I found the problem. This isn't an issue with a misunderstanding or an incorrect interpretation in the fix for #3997 . Rather, this is merely a failure on our part to update the MetadataConstraint, as well as it revealing a problem that MetaDataTableScanner was still using a very old schema and doing some manual parsing of the log entry's value instead of going through LogEntry, and it just so happened to still work until #3997 stopped using the value. (The parsing code in MetaDataTableScanner seems to be trying to parse a version of log entries that stored multiple log files inside the same value delimited by `|` and `;` characters, instead of being put in separate columns.) Although I proactively reverted #3997, it can be re-applied and immediately followed by this fix. I will change the misunderstanding that resulted in #3997 using "logReference" instead of "filePath". They are, in fact, paths, so I will change the variable and method names and comments back to what they were before when I re-apply that patch with the fix for this. I'm testing things locally, so I will have that up shortly as 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
