wjsl opened a new pull request, #1120: URL: https://github.com/apache/fluo/pull/1120
Rough cut of this. Needs to fix documentation, and add assertion to the tests. Open to suggestions on better test cases. This PR adds support for the following: 1. Specifying authentication tokens to use with the `FluoClient` on construction. 2. Specifying authentication tokens to use with a `Snapshot` when performing a read. I also think it makes sense to ensure the fluo table has the constraint set on it so unreadable data can't be written. Currently the behavior I'd see in my simple test is Fluo would kick back an error about a failed transaction because it couldn't read the data back. I think the better behavior (as Keith suggested) would be avoid any confusion in that case and kick back a "hey this data violates a table constraint due to its visibilities" error. -- 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]
