Skip to content

Conversation

@nastra
Copy link
Contributor

@nastra nastra commented Oct 6, 2021

No description provided.

dimas-b
dimas-b previously approved these changes Oct 6, 2021
@nastra nastra linked an issue Oct 6, 2021 that may be closed by this pull request
@nastra nastra force-pushed the spark-sql-syntax-improvements branch from dd1adf3 to 407e607 Compare October 7, 2021 06:00
@codecov
Copy link

codecov bot commented Oct 7, 2021

Codecov Report

Merging #2273 (a2d025f) into main (d835229) will increase coverage by 0.60%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #2273      +/-   ##
============================================
+ Coverage     83.76%   84.37%   +0.60%     
- Complexity     1843     1856      +13     
============================================
  Files           237      237              
  Lines         10461    10464       +3     
  Branches        734      734              
============================================
+ Hits           8763     8829      +66     
+ Misses         1393     1330      -63     
  Partials        305      305              
Flag Coverage Δ
java 84.04% <100.00%> (+0.68%) ⬆️
javascript 86.61% <ø> (ø)
python 87.13% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...execution/datasources/v2/AssignReferenceExec.scala 96.42% <100.00%> (+96.42%) ⬆️
...nessie/versioned/persist/tx/TxDatabaseAdapter.java 74.15% <0.00%> (-0.64%) ⬇️
...er/extensions/NessieSparkSqlExtensionsParser.scala 46.29% <0.00%> (+1.85%) ⬆️
...ser/extensions/NessieSqlExtensionsAstBuilder.scala 94.82% <0.00%> (+10.34%) ⬆️
...ources/v2/NessieExtendedDataSourceV2Strategy.scala 98.66% <0.00%> (+17.33%) ⬆️
...atalyst/plans/logical/AssignReferenceCommand.scala 87.50% <0.00%> (+87.50%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d835229...a2d025f. Read the comment docs.

@nastra nastra requested a review from dimas-b October 7, 2021 08:45
snazy
snazy previously approved these changes Oct 7, 2021
Copy link
Member

@snazy snazy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think, the change is legit and leans more towards "naturally spoken language" idiom.

@nastra nastra force-pushed the spark-sql-syntax-improvements branch from 407e607 to a2d025f Compare October 7, 2021 09:40
@nastra nastra requested a review from snazy October 7, 2021 09:49
@nastra nastra merged commit 42bbcf9 into projectnessie:main Oct 7, 2021
@nastra nastra deleted the spark-sql-syntax-improvements branch October 7, 2021 11:33
XN137 added a commit to XN137/nessie-demos that referenced this pull request May 19, 2022
- upgrade to iceberg v0.13.1
- upgrade to flink v1.14.4
- update notebooks to new sql grammar
  (see projectnessie/nessie#2273)
- update notebooks to new nessie cli
- update noteboosk to new nessie python api
- avoid silent exception handling
- apply delta workaround
  (see projectnessie/nessie#3552)
XN137 added a commit to XN137/nessie-demos that referenced this pull request May 20, 2022
- upgrade to iceberg 0.13.1
- upgrade to flink 1.13.6
- update notebooks to new sql grammar
  (see projectnessie/nessie#2273)
- adjust notebooks to new nessie cli
- adjust notebooks to new nessie python api
- avoid silent exception handling
- apply delta workaround
  (see projectnessie/nessie#3552)
XN137 added a commit to XN137/nessie-demos that referenced this pull request May 23, 2022
- upgrade to iceberg 0.13.1
- upgrade to flink 1.13.6
- update notebooks to new sql grammar
  (see projectnessie/nessie#2273)
- adjust notebooks to new nessie cli
- adjust notebooks to new nessie python api
- adjust notebooks to no longer claim branches have the same
  hash after merging
- avoid silent exception handling
- apply workaround for delta demo
  (for projectnessie/nessie#3552)
- apply workaround for hive demo
  (for apache/iceberg#4509)
XN137 added a commit to XN137/nessie-demos that referenced this pull request May 23, 2022
- upgrade to iceberg 0.13.1
- upgrade to flink 1.13.6
- update notebooks to new sql grammar
  (see projectnessie/nessie#2273)
- adjust notebooks to new nessie cli
- adjust notebooks to new nessie python api
- adjust notebooks to no longer claim branches have the same
  hash after merging
- avoid silent exception handling
- apply workaround for delta demo
  (for projectnessie/nessie#3552)
- apply workaround for hive demo
  (for apache/iceberg#4509)
XN137 added a commit to projectnessie/nessie-demos that referenced this pull request May 24, 2022
- upgrade to iceberg 0.13.1
- upgrade to flink 1.13.6
- update notebooks to new sql grammar
  (see projectnessie/nessie#2273)
- adjust notebooks to new nessie cli
- adjust notebooks to new nessie python api
- adjust notebooks to no longer claim branches have the same
  hash after merging
- avoid silent exception handling
- apply workaround for delta demo
  (for projectnessie/nessie#3552)
- apply workaround for hive demo
  (for apache/iceberg#4509)

closes #260
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.

Rename AS to FROM/TO in SQL Extensions

3 participants