Skip to content

#19 Initial CTE implementation#706

Merged
joocer merged 7 commits into
mainfrom
FEATURE/#19
Dec 7, 2022
Merged

#19 Initial CTE implementation#706
joocer merged 7 commits into
mainfrom
FEATURE/#19

Conversation

@joocer
Copy link
Copy Markdown
Member

@joocer joocer commented Dec 7, 2022

Release also includes fixes relating to new version of MyPy

@joocer joocer changed the title Feature/#19 #19 Initial CTE implementation Dec 7, 2022
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Dec 7, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 7, 2022

Codecov Report

Merging #706 (6a540b4) into main (27a4fc4) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #706      +/-   ##
==========================================
+ Coverage   96.10%   96.11%   +0.01%     
==========================================
  Files         160      160              
  Lines        6266     6288      +22     
==========================================
+ Hits         6022     6044      +22     
  Misses        244      244              
Impacted Files Coverage Δ
opteryx/managers/planner/logical/builders.py 97.93% <ø> (ø)
opteryx/managers/planner/temporal.py 100.00% <ø> (ø)
opteryx/operators/show_create_node.py 94.28% <ø> (ø)
opteryx/third_party/distogram/__init__.py 100.00% <ø> (ø)
opteryx/third_party/hyperloglog/hyperloglog.py 100.00% <ø> (ø)
opteryx/utils/dates.py 86.25% <ø> (ø)
tests/sql_battery/test_run_only_battery.py 100.00% <ø> (ø)
opteryx/connectors/mongodb_connector.py 100.00% <100.00%> (ø)
opteryx/functions/string_functions.py 90.56% <100.00%> (ø)
opteryx/managers/expression/__init__.py 99.14% <100.00%> (+<0.01%) ⬆️
... and 13 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ghost
Copy link
Copy Markdown

ghost commented Dec 7, 2022

👇 Click on the image for a new way to code review
  • Make big changes easier — review code in small groups of related files

  • Know where to start — see the whole change at a glance

  • Take a code tour — explore the change with an interactive tour

  • Make comments and review — all fully sync’ed with github

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

)
quoted_strings = re.compile(
r'(?:[^"\s]*"(?:\\.|[^"])*"[^"\s]*)+|(?:[^\'\s]*\'(?:\\.|[^\'])*\'[^\'\s]*)+|[^\s]+'
r'(?:[^"\s]*"(?:\\.|[^"])*")+|(?:[^\'\s]*\'(?:\\.|[^\'])*\')+|[^\s]+'

Check failure

Code scanning / CodeQL

Inefficient regular expression

This part of the regular expression may cause exponential backtracking on strings starting with '"' and containing many repetitions of '\\!'.
)
quoted_strings = re.compile(
r'(?:[^"\s]*"(?:\\.|[^"])*"[^"\s]*)+|(?:[^\'\s]*\'(?:\\.|[^\'])*\'[^\'\s]*)+|[^\s]+'
r'(?:[^"\s]*"(?:\\.|[^"])*")+|(?:[^\'\s]*\'(?:\\.|[^\'])*\')+|[^\s]+'

Check failure

Code scanning / CodeQL

Inefficient regular expression

This part of the regular expression may cause exponential backtracking on strings starting with ''' and containing many repetitions of '\\&'.
@joocer joocer merged commit 0947036 into main Dec 7, 2022
@joocer joocer deleted the FEATURE/#19 branch December 7, 2022 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants