Issue 12767053: add google api support & other features

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Issue 12767053: add google api support & other features (Closed)

Can't Edit
Can't Publish+Mail
Start Review

Created:
10 years, 8 months ago by stephenz-oti

Modified:
10 years, 8 months ago

Reviewers:
dominic-mlab

Base URL:
http://analysis.m-lab.googlecode.com/git/

Visibility:
Public.

More Reviews

Patch Set 1 #

Total comments: 3

Created: 10 years, 8 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+348 lines, -95 lines)			Patch
M	queryview/README	View	1 chunk	+2 lines, -2 lines	0 comments	Download
M	queryview/create_graphs.sh	View	2 chunks	+4 lines, -4 lines	0 comments	Download
M	queryview/queryview.py	View	11 chunks	+342 lines, -89 lines	3 comments	Download

Messages

Total messages: 4

Expand All Messages | Collapse All Messages

stephenz-oti

This patch covers these features: * adds direct support for bigquery via google api. * ...

10 years, 8 months ago (2013-08-26 03:37:16 UTC) #1

dominic-mlab

LGTM move to github? https://codereview.appspot.com/12767053/diff/1/queryview/queryview.py File queryview/queryview.py (right): https://codereview.appspot.com/12767053/diff/1/queryview/queryview.py#newcode10 queryview/queryview.py:10: import httplib2 these should be ...

10 years, 8 months ago (2013-08-26 16:50:10 UTC) #2

stephenz-oti

On 2013/08/26 16:50:10, dominic-mlab wrote: > LGTM > > move to github? > No problem. ...

10 years, 8 months ago (2013-08-26 18:02:36 UTC) #3

dominic-mlab

10 years, 8 months ago (2013-08-26 19:03:08 UTC) #4

it might be worth splitting the analysis repo into separate pieces but we
can do that later. thanks for importing.

i'll remove the old repo and add something to the main project home page.

Dominic Hamon | Measurement Lab
http://measurementlab.net

On Mon, Aug 26, 2013 at 11:02 AM, <soltesz@opentechinstitute.org> wrote:

> On 2013/08/26 16:50:10, dominic-mlab wrote:
>
>> LGTM
>>
>
>  move to github?
>>
>
>
> No problem.  I can do a simple init and import for m-lab.analysis to
> m-lab/analysis.
>
> Is there anything special todo on the old repo to signal the move?
>
>
https://codereview.appspot.**com/12767053/<https://codereview.appspot.com/127...
>

Expand All Messages | Collapse All Messages