[NCBC-2543] QueryIndexManager fails to create indexes in Linq2Couchbase integration tests - Couchbase database

XML

Word

Printable

Details

Type: Bug
Status: Resolved
Priority: Major
Resolution: Duplicate
Affects Version/s: 3.0.1
Fix Version/s: 3.3.0
Component/s: library
Labels:
- n1ql

Story Points:
1

Description

As an example, this method is being called:

await manager.CreateIndexAsync("beer-sample", "brewery_id", "brewery_id");

And this is the result:

Couchbase.Core.Exceptions.ParsingFailureException : syntax error - at - [3000]

   at Couchbase.Query.QueryResultExtensions.ThrowExceptionOnError[T](IQueryResult`1 result, QueryErrorContext context) in C:\Users\bburnett\source\repos\couchbase-net-client\src\Couchbase\Query\QueryResultExtensions.cs:line 56

   at Couchbase.Query.QueryClient.ExecuteQuery[T](QueryOptions options, ITypeSerializer serializer) in C:\Users\bburnett\source\repos\couchbase-net-client\src\Couchbase\Query\QueryClient.cs:line 204

   at Couchbase.Query.QueryClient.QueryAsync[T](String statement, QueryOptions options) in C:\Users\bburnett\source\repos\couchbase-net-client\src\Couchbase\Query\QueryClient.cs:line 67

   at Couchbase.Management.Query.QueryIndexManager.CreateIndexAsync(String bucketName, String indexName, IEnumerable`1 fields, CreateQueryIndexOptions options) in C:\Users\bburnett\source\repos\couchbase-net-client\src\Couchbase\Management\Query\QueryIndexManger.cs:line 67

   at Couchbase.Linq.IntegrationTests.N1QlTestBase.EnsureIndexExists(IBucket bucket, String indexName, String[] fields) in C:\Users\bburnett\source\repos\Linq2Couchbase\Src\Couchbase.Linq.IntegrationTests\N1QLTestBase.cs:line 18

   at Couchbase.Linq.IntegrationTests.QueryTests.JoinTests_InnerJoin_IndexJoin() in C:\Users\bburnett\source\repos\Linq2Couchbase\Src\Couchbase.Linq.IntegrationTests\QueryTests.cs:line 788

   at NUnit.Framework.Internal.TaskAwaitAdapter.GenericAdapter`1.BlockUntilCompleted()

   at NUnit.Framework.Internal.MessagePumpStrategy.NoMessagePumpStrategy.WaitForCompletion(AwaitAdapter awaitable)

   at NUnit.Framework.Internal.AsyncToSyncAdapter.Await(Func`1 invoke)

   at NUnit.Framework.Internal.Commands.TestMethodCommand.RunTestMethod(TestExecutionContext context)

   at NUnit.Framework.Internal.Commands.TestMethodCommand.Execute(TestExecutionContext context)

   at NUnit.Framework.Internal.Commands.BeforeAndAfterTestCommand.<>c__DisplayClass1_0.<Execute>b__0()

   at NUnit.Framework.Internal.Commands.BeforeAndAfterTestCommand.RunTestMethodInThreadAbortSafeZone(TestExecutionContext context, Action action)

Likely cause: Not escaping the - character in the bucket name.

Attachments

Issue Links

duplicates

NCBC-3107 IQueryIndexManager doesn't accept bucket names already in backticks

Resolved

relates to

NCBC-2955 CreateIndexAsync doesn't escape bucket names with dashes

Resolved

Gerrit Reviews

- Issue Only
- Show All Reviews
- Show Open Reviews
- Show All Issues
- Show Open Issues

No reviews matched the request. Check your Options in the drop-down menu of this sections header.

Activity

Ascending order - Click to sort in descending order

Brant Burnett created issue - 08/Jun/20 7:54 PM

Brant Burnett added a comment - 08/Jun/20 7:56 PM

Confirmed, passing "`beer-sample`" as the bucket name instead of "beer-sample" is a valid workaround.

Brant Burnett added a comment - 08/Jun/20 7:56 PM Confirmed, passing "`beer-sample`" as the bucket name instead of "beer-sample" is a valid workaround.

Matt Ingenthron made changes - 26/Jun/20 2:46 PM

Field	Original Value	New Value
Rank		Ranked higher

Jeff Morris made changes - 29/Jun/20 8:41 AM

Status

New [ 10003 ]

Open [ 1 ]

Jeff Morris made changes - 14/Jul/20 11:03 AM

Fix Version/s		3.0.4 [ 17014 ]
Fix Version/s	3.0.3 [ 16913 ]

Jeff Morris made changes - 04/Aug/20 8:40 PM

Fix Version/s		3.0.5 [ 17033 ]
Fix Version/s	3.0.4 [ 17014 ]

Jeff Morris made changes - 28/Aug/20 4:40 PM

Fix Version/s		backlog-3.0 [ 16772 ]
Fix Version/s	3.0.5 [ 17033 ]

Jeff Morris made changes - 28/Aug/20 4:40 PM

Assignee

Jeff Morris [ jmorris ]

Jeff Morris added a comment - 28/Aug/20 4:40 PM

This need to be brought up as an agenda item and discussed with the team as changing it should be done by all SDK's if it is done.

Jeff Morris added a comment - 28/Aug/20 4:40 PM This need to be brought up as an agenda item and discussed with the team as changing it should be done by all SDK's if it is done.

Richard Ponton added a comment - 02/Sep/20 10:06 AM

Group decision: SDK should surround everything in backticks.

This will have to be called out in the release notes, as it will then break for anyone using the workaround.

Richard Ponton added a comment - 02/Sep/20 10:06 AM Group decision: SDK should surround everything in backticks. This will have to be called out in the release notes, as it will then break for anyone using the workaround.

Jeff Morris made changes - 18/Nov/20 7:01 PM

Fix Version/s		3.1.0 [ 16741 ]
Fix Version/s	backlog-3.0 [ 16772 ]

Jeff Morris added a comment - 18/Nov/20 7:33 PM

CREATE INDEX brewery_id ON beer-sample(brewery_id) USING GSI WITH {"defer_build":False};

    "code": 3000,

    "msg": "syntax error - at -",

    "query": "CREATE INDEX brewery_id ON beer-sample(brewery_id) USING GSI WITH {\"defer_build\":False};"

Jeff Morris added a comment - 18/Nov/20 7:33 PM CREATE INDEX brewery_id ON beer-sample(brewery_id) USING GSI WITH {"defer_build":False}; [ { "code": 3000, "msg": "syntax error - at -", "query": "CREATE INDEX brewery_id ON beer-sample(brewery_id) USING GSI WITH {\"defer_build\":False};" } ]

Jeff Morris made changes - 30/Nov/20 12:01 PM

Fix Version/s		backlog-3.0 [ 16772 ]
Fix Version/s	3.1.0 [ 16741 ]

Jeff Morris made changes - 30/Jul/21 3:43 PM

Fix Version/s		3.2.1 [ 17902 ]
Fix Version/s	backlog-3.0 [ 16772 ]

Jeff Morris made changes - 30/Jul/21 3:43 PM

Assignee

Jeff Morris [ jmorris ]

Jeff Morris made changes - 30/Jul/21 3:43 PM

Status

Open [ 1 ]

In Progress [ 3 ]

Jeff Morris made changes - 08/Sep/21 6:25 PM

Resolution		Fixed [ 1 ]
Status	In Progress [ 3 ]	Resolved [ 5 ]

Jeff Morris made changes - 08/Sep/21 8:07 PM

Fix Version/s		3.2.2 [ 18043 ]
Fix Version/s	3.2.1 [ 17902 ]

Jeff Morris made changes - 08/Sep/21 8:07 PM

Resolution	Fixed [ 1 ]
Status	Resolved [ 5 ]	Reopened [ 4 ]

Jeff Morris made changes - 17/Sep/21 12:17 PM

Fix Version/s		3.2.3 [ 18050 ]
Fix Version/s	3.2.2 [ 18043 ]

Jeff Morris added a comment - 07/Oct/21 11:20 AM

Reopening as patch causes a regression when SELECT-BUCKET is called.

Jeff Morris added a comment - 07/Oct/21 11:20 AM Reopening as patch causes a regression when SELECT-BUCKET is called.

Jeff Morris made changes - 08/Oct/21 12:26 PM

Fix Version/s		3.2.4 [ 18075 ]
Fix Version/s	3.2.3 [ 18050 ]

Jeff Morris made changes - 11/Oct/21 11:32 AM

Link

This issue relates to ~~NCBC-2955~~ [ ~~NCBC-2955~~ ]

Jeff Morris made changes - 01/Nov/21 8:35 AM

Fix Version/s		3.2.5 [ 18102 ]
Fix Version/s	3.2.4 [ 18075 ]

Jeff Morris made changes - 07/Dec/21 3:32 PM

Fix Version/s		3.2.6 [ 18208 ]
Fix Version/s	3.2.5 [ 18102 ]

Jeff Morris made changes - 12/Jan/22 12:13 PM

Fix Version/s		3.2.7 [ 18230 ]
Fix Version/s	3.2.6 [ 18208 ]

Jeff Morris made changes - 01/Feb/22 6:57 PM

Fix Version/s		3.2.8 [ 18237 ]
Fix Version/s	3.2.7 [ 18230 ]

Jeff Morris made changes - 24/Feb/22 10:44 AM

Fix Version/s		3.2.9 [ 18306 ]
Fix Version/s	3.2.8 [ 18237 ]

Jeff Morris made changes - 04/Apr/22 3:06 PM

Fix Version/s		3.3.0 [ 18323 ]
Fix Version/s	3.2.9 [ 18306 ]

Jeff Morris added a comment - 20/Apr/22 4:31 PM

~~NCBC-3107~~

Jeff Morris added a comment - 20/Apr/22 4:31 PM NCBC-3107

Jeff Morris made changes - 20/Apr/22 4:31 PM

Resolution		Duplicate [ 3 ]
Status	Reopened [ 4 ]	Resolved [ 5 ]

Jeff Morris made changes - 20/Apr/22 4:31 PM

Link

This issue duplicates ~~NCBC-3107~~ [ ~~NCBC-3107~~ ]

Matt Ingenthron made changes - 21/Dec/22 10:44 PM

Workflow

Couchbase SDK Workflow [ 171261 ]

Couchbase SDK Workflow with Review [ 249006 ]

People

Assignee:: Jeff Morris

Reporter:: Brant Burnett

Votes:: 0 Vote for this issue

Watchers:: 3 Start watching this issue

Dates

Created:: 08/Jun/20 7:54 PM

Updated:: 20/Apr/22 4:31 PM

Resolved:: 20/Apr/22 4:31 PM

Gerrit Reviews

There are no open Gerrit changes

Show There are 2 closed Gerrit changes

Hide There are 2 closed Gerrit changes

NCBC-2543: escape the bucket name with backtics: Gerrit Review:

Revert "NCBC-2543: escape the bucket name with backtics": Gerrit Review:

PagerDuty

Couchbase .NET client library

Details

Description

Attachments

Issue Links

Gerrit Reviews

Activity

People

Dates

Gerrit Reviews

PagerDuty