PrimeIntellect/SWE-rebench-V2-Filtered-Easy-Verified huggingface dataset:
Repository: kayak/pypika (python)
Issue: Extraneous quotes when using passing Field to between
[System Environment]

Windows 10
Python: 3.6.4
PyPika: 0.15.7

[Overview]
When doing a join based on a table field's value being between values derived from another field (or set of fields), the `start` and `end` portions get wrapped in quotes. 

```
t1 = Table('Table1', alias='t1')
t2 = Table('Table2', alias='t2')

Query.from_(t1).join(t2).on(t1.Value.between(t2.Start, t2.End)).select(t1.value).get_sql()
```

yields:

`'SELECT t1.value FROM Table1 t1 JOIN Table2 t2 ON t1.Value BETWEEN "t2"."Start" AND "t2"."End"'`


Looking at `BetweenCriterion` it seems like, if `self.start` and/or `self.stop` are instances of `Term` then `get_sql(**kwargs)` should be called (as is done with `self.term`). Unless this is intended behavior? 
Pull request description: Fixed usage of quote character in BETWEEN criterian
Fixes #168 fixed get_sql in BetweenCriterian to stringify fields passing kwargs

Patch:
diff --git a/pypika/queries.py b/pypika/queries.py
index bf831fc..51652f3 100644
--- a/pypika/queries.py
+++ b/pypika/queries.py
@@ -693,6 +693,8 @@ class QueryBuilder(Selectable, Term):
         return hash(self.alias) + sum(hash(clause) for clause in self._from)
 
     def get_sql(self, with_alias=False, subquery=False, **kwargs):
+        kwargs.setdefault('quote_char', self.quote_char)
+
         if not (self._selects or self._insert_table or self._delete_from or self._update_table):
             return ''
         if self._insert_table and not (self._selects or self._values):
diff --git a/pypika/terms.py b/pypika/terms.py
index 13ddafe..cf9d065 100644
--- a/pypika/terms.py
+++ b/pypika/terms.py
@@ -494,8 +494,8 @@ class BetweenCriterion(Criterion):
         # FIXME escape
         return "{term} BETWEEN {start} AND {end}".format(
             term=self.term.get_sql(**kwargs),
-            start=self.start,
-            end=self.end,
+            start=self.start.get_sql(**kwargs),
+            end=self.end.get_sql(**kwargs),
         )
 
     def fields(self):


Test patch:
diff --git a/pypika/tests/test_selects.py b/pypika/tests/test_selects.py
index 8730d70..c1d673f 100644
--- a/pypika/tests/test_selects.py
+++ b/pypika/tests/test_selects.py
@@ -778,3 +778,15 @@ class SubqueryTests(unittest.TestCase):
             'WITH an_alias AS (SELECT "fizz" FROM "efg") '
             'SELECT * FROM "abc" JOIN an_alias ON "an_alias"."fizz"="abc"."buzz"',
             str(test_query))
+
+        
+class QuoteTests(unittest.TestCase):
+    def test_extraneous_quotes(self):
+        t1 = Table('table1', alias='t1')
+        t2 = Table('table2', alias='t2')
+
+        query = Query.from_(t1).join(t2).on(t1.Value.between(t2.start, t2.end)).select(t1.value)
+
+        self.assertEqual('SELECT t1.value FROM table1 t1 '
+                         'JOIN table2 t2 ON t1.Value '
+                         'BETWEEN t2.start AND t2.end', query.get_sql(quote_char=None))
Repository: googleapis/api-linter (go)
Issue: C# namespace capitalization for versions is incorrect
The linter just complained about this option:

```proto
option csharp_namespace = "Google.Cloud.SecurityCenter.V1P1Beta1";
````

it thinks it should be:

```proto
option csharp_namespace = "Google.Cloud.SecurityCenter.V1p1Beta1";
````

... which is actually the bug I was trying to fix. In general, the PascalCasing of C# names considers numbers to be word separators, so we'd have `Http2Request` for example, rather than `Http2request`.
Pull request description: [AIP-191] Handle point versions correctly in C#.
Fixes #557.

Patch:
diff --git a/rules/aip0191/csharp_namespace.go b/rules/aip0191/csharp_namespace.go
index 1f10b90..436e062 100644
--- a/rules/aip0191/csharp_namespace.go
+++ b/rules/aip0191/csharp_namespace.go
@@ -44,7 +44,11 @@ var csharpNamespace = &lint.FileRule{
 				wantSegment := csharpVersionRegexp.ReplaceAllStringFunc(
 					strcase.UpperCamelCase(segment),
 					func(s string) string {
-						stability := csharpVersionRegexp.FindStringSubmatch(s)[1]
+						point := csharpVersionRegexp.FindStringSubmatch(s)[1]
+						if point != "" {
+							s = strings.ReplaceAll(s, point, strings.ToUpper(point))
+						}
+						stability := csharpVersionRegexp.FindStringSubmatch(s)[2]
 						return strings.ReplaceAll(s, stability, strings.Title(stability))
 					},
 				)
@@ -64,4 +68,4 @@ var csharpNamespace = &lint.FileRule{
 }
 
 var csharpValidChars = regexp.MustCompile("^[A-Za-z0-9.]+$")
-var csharpVersionRegexp = regexp.MustCompile("[0-9]+(alpha|beta)")
+var csharpVersionRegexp = regexp.MustCompile("[0-9]+(p[0-9]+)?(alpha|beta)")


Test patch:
diff --git a/rules/aip0191/csharp_namespace_test.go b/rules/aip0191/csharp_namespace_test.go
index a24ee3c..016b0b4 100644
--- a/rules/aip0191/csharp_namespace_test.go
+++ b/rules/aip0191/csharp_namespace_test.go
@@ -27,8 +27,9 @@ func TestCsharpNamespace(t *testing.T) {
 		CsharpNamespace string
 		problems        testutils.Problems
 	}{
-		{"Valid", "Google.Example.V1", testutils.Problems{}},
-		{"ValidBeta", "Google.Example.V1Beta1", testutils.Problems{}},
+		{"Valid", "Google.Example.V1", nil},
+		{"ValidBeta", "Google.Example.V1Beta1", nil},
+		{"ValidPointBeta", "Google.Example.V1P1Beta1", nil},
 		{"InvalidBadChars", "Google:Example:V1", testutils.Problems{{Message: "Invalid characters"}}},
 		{"Invalid", "google.example.v1", testutils.Problems{{
 			Suggestion: fmt.Sprintf("option csharp_namespace = %q;", "Google.Example.V1"),
@@ -39,6 +40,9 @@ func TestCsharpNamespace(t *testing.T) {
 		{"InvalidBeta", "Google.Example.V1beta1", testutils.Problems{{
 			Suggestion: fmt.Sprintf("option csharp_namespace = %q;", "Google.Example.V1Beta1"),
 		}}},
+		{"InvalidPointBeta", "Google.Example.V1p1beta1", testutils.Problems{{
+			Suggestion: fmt.Sprintf("option csharp_namespace = %q;", "Google.Example.V1P1Beta1"),
+		}}},
 	} {
 		t.Run(test.name, func(t *testing.T) {
 			f := testutils.ParseProto3Tmpl(t, `