Couchbase .NET client library

Compile time error in MemcachedTest project - VBucket constructor now takes 3 parameters

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None

Description

VBucket constructor now takes a third parameter - index.
The class EnyimMemcached\MemcachedTest\VBucketTest.cs initializes VBucket with only two arguments, that leads to compile time error.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Saakshi Manocha added a comment -
- Forked the reporsitory from Couchbase\Enyimmemcached.
- Added third parameter 'index' while initializing VBucket.
- Committed and pushed changes into forked repository.
- Send a pull request to master repository for review and merge.

Change done:
- File changed: EnyimMemcached\MemcachedTest\VBucketTest.cs
- Replaced the following:

private VBucket[] buckets = new[]
{
new VBucket(0, new[] {1, 2}),
new VBucket(1, new[] {2, 0}),
new VBucket(2, new[] {1, -1}),
new VBucket(1, new[] {2, 0})
};


with

private VBucket[] buckets = new[]
{
new VBucket(0, new[] {1, 2}, 0),
new VBucket(1, new[] {2, 0}, 1),
new VBucket(2, new[] {1, -1}, 2),
new VBucket(1, new[] {2, 0}, 3)
};

Show
Saakshi Manocha added a comment - - Forked the reporsitory from Couchbase\Enyimmemcached. - Added third parameter 'index' while initializing VBucket. - Committed and pushed changes into forked repository. - Send a pull request to master repository for review and merge. Change done: - File changed: EnyimMemcached\MemcachedTest\VBucketTest.cs - Replaced the following: private VBucket[] buckets = new[] { new VBucket(0, new[] {1, 2}), new VBucket(1, new[] {2, 0}), new VBucket(2, new[] {1, -1}), new VBucket(1, new[] {2, 0}) }; with private VBucket[] buckets = new[] { new VBucket(0, new[] {1, 2}, 0), new VBucket(1, new[] {2, 0}, 1), new VBucket(2, new[] {1, -1}, 2), new VBucket(1, new[] {2, 0}, 3) };
Saakshi Manocha made changes -
Field Original Value New Value
Status Open [ 1 ] In Progress [ 3 ]
Hide
Permalink
Saakshi Manocha added a comment -
change merged with master repository
Show
Saakshi Manocha added a comment - change merged with master repository
Saakshi Manocha made changes -
Status In Progress [ 3 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]

People

  • Assignee:
    Saakshi Manocha
    Reporter:
    Saakshi Manocha
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: