Skip to content

Docs: Added Upsolver to vendor list#10096

Merged
nastra merged 1 commit into
apache:mainfrom
Upsolver:add-upsolver-to-vendors
Apr 9, 2024
Merged

Docs: Added Upsolver to vendor list#10096
nastra merged 1 commit into
apache:mainfrom
Upsolver:add-upsolver-to-vendors

Conversation

@jasonf20
Copy link
Copy Markdown
Contributor

@jasonf20 jasonf20 commented Apr 7, 2024

This PR simply adds Upsolver to the vendor list docs page.

@github-actions github-actions Bot added the docs label Apr 7, 2024
Copy link
Copy Markdown
Collaborator

@bitsondatadev bitsondatadev left a comment

Choose a reason for hiding this comment

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

I would suggest you remove a lot of the redundant phrasing that is implied by this being a service built on Iceberg.

Comment thread site/docs/vendors.md Outdated
@jasonf20 jasonf20 requested a review from bitsondatadev April 8, 2024 11:46
Copy link
Copy Markdown
Collaborator

@bitsondatadev bitsondatadev left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @jasonf20

Copy link
Copy Markdown
Collaborator

@bitsondatadev bitsondatadev left a comment

Choose a reason for hiding this comment

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

Update: please do remove utm codes please.

Comment thread site/docs/vendors.md Outdated
Copy link
Copy Markdown
Collaborator

@bitsondatadev bitsondatadev left a comment

Choose a reason for hiding this comment

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

Thanks for the update @jasonf20. @nastra, @RussellSpitzer this all looks good to me if could merge please?

@nastra nastra merged commit a351e22 into apache:main Apr 9, 2024
@jasonf20 jasonf20 deleted the add-upsolver-to-vendors branch April 11, 2024 08:47
sasankpagolu pushed a commit to sasankpagolu/iceberg that referenced this pull request Oct 27, 2024
zachdisc pushed a commit to zachdisc/iceberg that referenced this pull request Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants