FWD » Bugs

04/21/2016

08:51 AM Bug #1706: temp-table buffer scoping

It may not be an issue any more because we have made changes recently to temp-table scoping and buffer processing. B... Greg Shah

08:37 AM Bug #1706: temp-table buffer scoping

I know this is a very old issue - is it still relevant?
How would we determine whether our code suffers from this ... Paul E

04/06/2016

05:18 AM Bug #3057: bad synchronization in AsyncRequestImpl

I agree, the @AsyncRequestImpl@ is too synchronized. I think is enough to set @volatile@ for the @complete@, @cancel... Constantin Asofiei

04/05/2016

03:39 PM Bug #3057: bad synchronization in AsyncRequestImpl

To duplicate the issue I used an asynchronous call to a (slow) web-service and @stop@/ @disconnect@ before it has the... Ovidiu Maxiniuc

03:21 PM Bug #3057: bad synchronization in AsyncRequestImpl

I found the following issue in @AsyncRequestImpl.java@: the class is (too) heavily synchronized.
The @synchronized... Ovidiu Maxiniuc

03:11 PM Bug #3057 (Closed): bad synchronization in AsyncRequestImpl

Ovidiu Maxiniuc

04/04/2016

06:53 PM Bug #2483 (Closed): Fix issues identified by static code analysis

Since all of the potential defects from this round of static code analysis that were deemed worthwhile to address hav... Eric Faulhaber

02:54 PM Bug #2483: Fix issues identified by static code analysis

Merged and committed to the trunk revision 10992.
Task branch 2483a archived Igor Skornyakov

02:45 PM Bug #2483: Fix issues identified by static code analysis

Great! Please merge to trunk. Eric Faulhaber

02:36 PM Bug #2483: Fix issues identified by static code analysis

Majic code conversion and build with the task branch 2483a finished OK. Igor Skornyakov

01:01 PM Bug #2483: Fix issues identified by static code analysis

Eric Faulhaber wrote:
> Was the change you made today (rev. 11001) to @ExpressionConversionWorker@ made before or ... Igor Skornyakov

12:57 PM Bug #2483: Fix issues identified by static code analysis

Igor Skornyakov wrote:
> Can I merge it to the trunk?
Was the change you made today (rev. 11001) to @ExpressionConv... Eric Faulhaber

12:22 PM Bug #2483: Fix issues identified by static code analysis

Only @tc_job_002@ test failed after 3 runs of the @main-regression@.
The task branch 2483a was rebased from the tr... Igor Skornyakov

10:53 AM Bug #2483: Fix issues identified by static code analysis

Constantin Asofiei wrote:
> > Constantin, AFAIK, @TemporaryAccountPool@ is not exercised in normal regression test... Igor Skornyakov

10:33 AM Bug #2483: Fix issues identified by static code analysis

Eric Faulhaber wrote:
> Constantin, AFAIK, @TemporaryAccountPool@ is not exercised in normal regression testing, cor... Constantin Asofiei

09:11 AM Bug #2483: Fix issues identified by static code analysis

Constantin, AFAIK, @TemporaryAccountPool@ is not exercised in normal regression testing, correct? In other words, is ... Eric Faulhaber

09:08 AM Bug #2483: Fix issues identified by static code analysis

Eric Faulhaber wrote:
> Igor Skornyakov wrote:
> > CTRL-C part passed OK.
> This was the first regression testing ... Igor Skornyakov

09:06 AM Bug #2483: Fix issues identified by static code analysis

Igor Skornyakov wrote:
> CTRL-C part passed OK.
This was the first regression testing item you mentioned. However, ... Eric Faulhaber

08:59 AM Bug #2483: Fix issues identified by static code analysis

Constantin Asofiei wrote:
> I'm not referring to the synchronization of the @TemporaryAccountPool.pool@ field. I'... Igor Skornyakov

08:34 AM Bug #2483: Fix issues identified by static code analysis

After the second run only tc_job_002 and tc_job_clock_004 tests failed.
Test restarted. Igor Skornyakov

08:24 AM Bug #2483: Fix issues identified by static code analysis

Igor Skornyakov wrote:
> Constantin Asofiei wrote:
> > Review for 2483a 10999:
> > # @TemporaryAccountPool@ - you ... Constantin Asofiei

07:29 AM Bug #2483: Fix issues identified by static code analysis

Constantin Asofiei wrote:
> Review for 2483a 10999:
> # @TemporaryAccountPool@ - you are removing the synchronizati... Igor Skornyakov

06:34 AM Bug #2483: Fix issues identified by static code analysis

Review for 2483a 10999:
# @TemporaryAccountPool@ - you are removing the synchronization for @shutDown@ and @poll@ an... Constantin Asofiei

04:20 AM Bug #2483: Fix issues identified by static code analysis

From the main part the following tests failed in the first run: tc_dc_slot_026 tc_job_002 tc_job_clock_004 tc_ap_vchr... Igor Skornyakov

04/03/2016

06:05 PM Bug #2483: Fix issues identified by static code analysis

CTRL-C part passed OK. Igor Skornyakov

05:00 PM Bug #2483: Fix issues identified by static code analysis

Eric Faulhaber wrote:
> Igor, assuming the others are OK with the changes and regression testing is successful, plea... Igor Skornyakov

04:58 PM Bug #2483: Fix issues identified by static code analysis

Code review 2483a/10999:
The changes look OK to me.
Greg, Constantin: please confirm the changes to your files ... Eric Faulhaber

04:47 PM Bug #2483: Fix issues identified by static code analysis

Eric Faulhaber wrote:
> True, but that's the case for @java.util.Map<K,V>.entrySet()@ as well (after which these API... Igor Skornyakov

04:44 PM Bug #2483: Fix issues identified by static code analysis

True, but that's the case for @java.util.Map<K,V>.entrySet()@ as well (after which these APIs were modelled). Eric Faulhaber

04:37 PM Bug #2483: Fix issues identified by static code analysis

Eric Faulhaber wrote:
> Igor Skornyakov wrote:
> > What's the point to use @HashSet<Map.Entry<K, V>>@ instead of @H... Igor Skornyakov

04:30 PM Bug #2483: Fix issues identified by static code analysis

Igor Skornyakov wrote:
> What's the point to use @HashSet<Map.Entry<K, V>>@ instead of @HashMap<K,V>@?
The API in t... Eric Faulhaber

03:08 PM Bug #2483: Fix issues identified by static code analysis

Fixed a found.
Committed to the task branch 2483a revision 10999.
Restarted regression testing. Igor Skornyakov

02:38 PM Bug #2483: Fix issues identified by static code analysis

Finished with the next round of issues.
Committed to the task branch 2483a revision 10998.
Started regression ... Igor Skornyakov

06:52 AM Bug #2483: Fix issues identified by static code analysis

Igor Skornyakov wrote:
> The Findbugs complains about 3 places in the @ScopedDictionary@ class (statement @foo.addA... Igor Skornyakov

06:37 AM Bug #2483: Fix issues identified by static code analysis

Igor Skornyakov wrote:
> Regarding previous note:
> What's the point to use @HashSet<Map.Entry<K, V>>@ instead of ... Igor Skornyakov

06:05 AM Bug #2483: Fix issues identified by static code analysis

Regarding previous note:
What's the point to use @HashSet<Map.Entry<K, V>>@ instead of @HashMap<K,V>@?
Thank you. Igor Skornyakov

05:58 AM Bug #2483: Fix issues identified by static code analysis

The Findbugs complains about 3 places in the @ScopedDictionary@ class (statement @foo.addAll(dictionary.entrySet())@... Igor Skornyakov

04/02/2016

04:24 PM Bug #2483: Fix issues identified by static code analysis

Eric Faulhaber wrote:
> If anyone has further contributions on any of the current items under discussion, please exp... Igor Skornyakov

04:20 PM Bug #2483: Fix issues identified by static code analysis

If anyone has further contributions on any of the current items under discussion, please express those ASAP. This inc... Eric Faulhaber

04/01/2016

02:37 PM Bug #2483: Fix issues identified by static code analysis

> > My review for 2483a rev 10995:
> > # @ExpressionConversionWorker.expressionType:1056@ - adding the @stype = expr... Greg Shah

11:58 AM Bug #2483: Fix issues identified by static code analysis

Constantin Asofiei wrote:
> Igor Skornyakov wrote:
> > # @AclRow.equals@ - non symmetric e.g. when comparing @AclRo... Igor Skornyakov

10:58 AM Bug #2483: Fix issues identified by static code analysis

Igor Skornyakov wrote:
> Hynek Cihlar wrote:
>
> > Well, in case the @owner@ is @null@ (which must never happen f... Eugenie Lyzenko

08:54 AM Bug #2483: Fix issues identified by static code analysis

Hynek Cihlar wrote:
> Well, in case the @owner@ is @null@ (which must never happen for overlay windows by the way)... Igor Skornyakov

08:50 AM Bug #2483: Fix issues identified by static code analysis

Constantin Asofiei wrote:
> My review for 2483a rev 10995:
> # @ExpressionConversionWorker.expressionType:1056@ - a... Igor Skornyakov

06:41 AM Bug #2483: Fix issues identified by static code analysis

Hynek Cihlar wrote:
> Constantin Asofiei wrote:
> > My review for 2483a rev 10995:
> > 4 @ConfigSyncManager@ - thi... Constantin Asofiei

06:35 AM Bug #2483: Fix issues identified by static code analysis

Constantin Asofiei wrote:
> My review for 2483a rev 10995:
> 4 @ConfigSyncManager@ - this class is not only client-... Hynek Cihlar

06:13 AM Bug #2483: Fix issues identified by static code analysis

My review for 2483a rev 10995:
# @ExpressionConversionWorker.expressionType:1056@ - adding the @stype = expressionTy... Constantin Asofiei

05:25 AM Bug #2483: Fix issues identified by static code analysis

Eric Faulhaber wrote:
> If you think any are serious, then yes. Based on your earlier assessment of the remaining fi... Igor Skornyakov

04:50 AM Bug #2483: Fix issues identified by static code analysis

Eric Faulhaber wrote:
> Code review 2483a/10992:
>
> Lots of good changes. Also lots of format problems. Some com... Igor Skornyakov

12:00 AM Bug #2483: Fix issues identified by static code analysis

Code review 2483a/10992:
Lots of good changes. Also lots of format problems. Some comments on substance:
* I do... Eric Faulhaber