On 2016/08/24 00:41:37, pjo wrote: > Is there anywhere in the src where we are ...
7 years, 8 months ago
(2016-08-24 00:58:55 UTC)
#4
On 2016/08/24 00:41:37, pjo wrote:
> Is there anywhere in the src where we are still using getColumnName? (other
> than the test that throws ISE)
>
> Thank you
no. I replace all using getColumnName in "database" repository
also I replace one line in "library" also
https://codereview.appspot.com/303380043/
LGTM. Thank you - - technology's compounding interest On Tue, Aug 23, 2016 at 5:58 ...
7 years, 8 months ago
(2016-08-24 01:00:46 UTC)
#5
LGTM. Thank you
-
- technology's compounding interest
On Tue, Aug 23, 2016 at 5:58 PM, <imysak@google.com> wrote:
> On 2016/08/24 00:41:37, pjo wrote:
>
>> Is there anywhere in the src where we are still using getColumnName?
>>
> (other
>
>> than the test that throws ISE)
>>
>
> Thank you
>>
>
> no. I replace all using getColumnName in "database" repository
> also I replace one line in "library" also
> https://codereview.appspot.com/303380043/
>
> https://codereview.appspot.com/306280043/
>
Issue 306280043: <#26352925 Using column aliases instead of pure column name
(Closed)
Created 7 years, 8 months ago by imysak
Modified 7 years, 7 months ago
Reviewers: pjo
Base URL:
Comments: 0