{"action":"created","issue":{"url":"https://api.github.com/repos/cockroachdb/cockroach/issues/7465","repository_url":"https://api.github.com/repos/cockroachdb/cockroach","labels_url":"https://api.github.com/repos/cockroachdb/cockroach/issues/7465/labels{/name}","comments_url":"https://api.github.com/repos/cockroachdb/cockroach/issues/7465/comments","events_url":"https://api.github.com/repos/cockroachdb/cockroach/issues/7465/events","html_url":"https://github.com/cockroachdb/cockroach/pull/7465","id":162169309,"number":7465,"title":"rfc: sql interleaved tables","user":{"login":"paperstreet","id":52528,"avatar_url":"https://avatars.githubusercontent.com/u/52528?v=3","gravatar_id":"","url":"https://api.github.com/users/paperstreet","html_url":"https://github.com/paperstreet","followers_url":"https://api.github.com/users/paperstreet/followers","following_url":"https://api.github.com/users/paperstreet/following{/other_user}","gists_url":"https://api.github.com/users/paperstreet/gists{/gist_id}","starred_url":"https://api.github.com/users/paperstreet/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/paperstreet/subscriptions","organizations_url":"https://api.github.com/users/paperstreet/orgs","repos_url":"https://api.github.com/users/paperstreet/repos","events_url":"https://api.github.com/users/paperstreet/events{/privacy}","received_events_url":"https://api.github.com/users/paperstreet/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"petermattis","id":6201335,"avatar_url":"https://avatars.githubusercontent.com/u/6201335?v=3","gravatar_id":"","url":"https://api.github.com/users/petermattis","html_url":"https://github.com/petermattis","followers_url":"https://api.github.com/users/petermattis/followers","following_url":"https://api.github.com/users/petermattis/following{/other_user}","gists_url":"https://api.github.com/users/petermattis/gists{/gist_id}","starred_url":"https://api.github.com/users/petermattis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/petermattis/subscriptions","organizations_url":"https://api.github.com/users/petermattis/orgs","repos_url":"https://api.github.com/users/petermattis/repos","events_url":"https://api.github.com/users/petermattis/events{/privacy}","received_events_url":"https://api.github.com/users/petermattis/received_events","type":"User","site_admin":false},"assignees":[{"login":"petermattis","id":6201335,"avatar_url":"https://avatars.githubusercontent.com/u/6201335?v=3","gravatar_id":"","url":"https://api.github.com/users/petermattis","html_url":"https://github.com/petermattis","followers_url":"https://api.github.com/users/petermattis/followers","following_url":"https://api.github.com/users/petermattis/following{/other_user}","gists_url":"https://api.github.com/users/petermattis/gists{/gist_id}","starred_url":"https://api.github.com/users/petermattis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/petermattis/subscriptions","organizations_url":"https://api.github.com/users/petermattis/orgs","repos_url":"https://api.github.com/users/petermattis/repos","events_url":"https://api.github.com/users/petermattis/events{/privacy}","received_events_url":"https://api.github.com/users/petermattis/received_events","type":"User","site_admin":false}],"milestone":null,"comments":27,"created_at":"2016-06-24T15:16:23Z","updated_at":"2016-07-05T14:42:15Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cockroachdb/cockroach/pulls/7465","html_url":"https://github.com/cockroachdb/cockroach/pull/7465","diff_url":"https://github.com/cockroachdb/cockroach/pull/7465.diff","patch_url":"https://github.com/cockroachdb/cockroach/pull/7465.patch"},"body":"Support interleaving the data for sql tables such that the rows alternate in the\r\nkv map. This allows a user to optimize reads on tables that are frequently\r\njoined.\r\n\r\nRendered preview: Preview at https://github.com/paperstreet/cockroach/blob/rfc_interleaved/docs/RFCS/sql_interleaved_tables.md\n\n<!-- Reviewable:start -->\n---\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"35\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/cockroachdb/cockroach/7465)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/cockroachdb/cockroach/issues/comments/230498447","html_url":"https://github.com/cockroachdb/cockroach/pull/7465#issuecomment-230498447","issue_url":"https://api.github.com/repos/cockroachdb/cockroach/issues/7465","id":230498447,"user":{"login":"petermattis","id":6201335,"avatar_url":"https://avatars.githubusercontent.com/u/6201335?v=3","gravatar_id":"","url":"https://api.github.com/users/petermattis","html_url":"https://github.com/petermattis","followers_url":"https://api.github.com/users/petermattis/followers","following_url":"https://api.github.com/users/petermattis/following{/other_user}","gists_url":"https://api.github.com/users/petermattis/gists{/gist_id}","starred_url":"https://api.github.com/users/petermattis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/petermattis/subscriptions","organizations_url":"https://api.github.com/users/petermattis/orgs","repos_url":"https://api.github.com/users/petermattis/repos","events_url":"https://api.github.com/users/petermattis/events{/privacy}","received_events_url":"https://api.github.com/users/petermattis/received_events","type":"User","site_admin":false},"created_at":"2016-07-05T14:42:15Z","updated_at":"2016-07-05T14:42:15Z","body":"\n\n\n\nReview status: 0 of 1 files reviewed at latest revision, 13 unresolved discussions, all commit checks successful.\n\n---\n\n*[docs/RFCS/sql_interleaved_tables.md, line 86 \\[r4\\]](https://reviewable.io:443/reviews/cockroachdb/cockroach/7465#-KLvV4S0KwhP8pMCr5qm:-KLvV4S1MDY5jhQGAmmK:1738770368) ([raw file](https://github.com/cockroachdb/cockroach/blob/65f27ea0f2926a1c1d39b1a10c51ad7732c429c5/docs/RFCS/sql_interleaved_tables.md#L86)):*\n> ```Markdown\n>     /customers/1/<customerID>/<interleavedTableSentinel>/orders/<indexID>/<orderID>/<familyID>/suffix\n> \n> Where `interleavedTableSentinel` is 109, which is selected as the largest value\n> ```\n\n109 is the largest integer that can be encoded in a single byte, but is there a reason to restrict ourselves to an integer here. Seems like you could encoded `0xff` (`EncodeNullDescending`) to guarantee a \"familyID\" for the interleaved table sentinel that is larger than any normal familyID. Is this not feasible for some reason?\n\n---\n\n\n*Comments from [Reviewable](https://reviewable.io:443/reviews/cockroachdb/cockroach/7465)*\n<!-- Sent from Reviewable.io -->\n"}}
{"actor":{"display_login":"petermattis"}}