Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DPDK Backend : Minor fixes #3674

Merged
merged 1 commit into from
Nov 8, 2022
Merged

DPDK Backend : Minor fixes #3674

merged 1 commit into from
Nov 8, 2022

Conversation

usha1830
Copy link
Contributor

@usha1830 usha1830 commented Nov 7, 2022

  1. Error if learner table exists with non-exact match keys
  2. Removed explicit from constructor with more than one parameters
  3. Typo in condition check in context json generation
    `4) Added new test

1) Error if learner table exists with non-exact match keys
2) Removed explicit from constructor with more than one paramater
3) Typo in condition check in context json generation
4) Added new test
@@ -126,7 +126,7 @@ void DpdkContextGenerator::CollectTablesAndSetAttributes() {
struct externAttributes externAttr;
externAttr.externalName = ed->controlPlaneName();
externAttr.externType = externTypeName;
if (externTypeName == "Counter" || "DirectCounter") {
if (externTypeName == "Counter" || externTypeName == "DirectCounter") {
Copy link
Contributor

Choose a reason for hiding this comment

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

should have caught this one earlier.

@usha1830
Copy link
Contributor Author

usha1830 commented Nov 7, 2022

Fedora build is failing in installing dependencies

error: RPC failed; HTTP 503 curl 22 The requested URL returned error: 503
fatal: expected 'packfile'
Error: Process completed with exit code 128

@mihaibudiu mihaibudiu merged commit 8ee2e65 into p4lang:main Nov 8, 2022
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