feat: dbeaver mysql compatibility, use statement and information_schema.tables

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?

This patch is to resolve compatibility issues with dbeaver client:

it tries to access some field like data_size in information_schema.tables
it uses use statement instead of modern use command which most other clients use

This patch fills these two gaps and make dbeaver client works for most cases. There is another statement to be supported: SHOW TABLE STATUS FROM public LIKE 'system_metrics' but it's not a blocking issue for our scenario. SHOW TABLE STATUS is tracked in #3354
This patch brings back USE statement support in #2022 as we finally found a real use-case for it. And it also makes current_schema modifiable and propagate to Session.
The data fields of information_schema.TABLES in this patch are all for placeholder. I haven't fill real data into these fields. TODO item is left to future enhancement.
Future work
I'm going refactor Session and QueryContext to use Arc<RwLock<>> for all modifiable fields, like time-zone, schema. This makes the change automatically propagate from QueryContext to Session. The idea of QueryContext::update_session is smart, but it's going to be difficult to maintain in future.
Checklist

[ ] I have written the necessary rustdoc comments.
[ ] I have added the necessary unit tests and integration tests.
[ ] This PR requires documentation updates.

Summary by CodeRabbit

New Features

Added support for the USE statement to switch the current database schema.
Introduced new fields like DATA_LENGTH, INDEX_LENGTH, MAX_DATA_LENGTH, and AVG_ROW_LENGTH in the schema definition.

Bug Fixes

Corrected schema handling in various functions to consistently use references, improving memory efficiency and performance.

Refactor

Updated multiple method calls to use references for schema parameters for more efficient data handling.
Transitioned current_schema in QueryContext to use ArcSwap<String> for thread-safe updates.

Documentation

Improved internal documentation of schema handling changes to assist developers with understanding the updates.

Tests

Modified tests to align with the updated schema handling and ensure consistent behavior across all functions.

I'm going refactor Session and QueryContext to use Arc<RwLock<>> for all modifiable fields, like time-zone, schema. This > makes the change automatically propagate from QueryContext to Session. The idea of QueryContext::update_session is > smart, but it's going to be difficult to maintain in future.

It may hurt performance, must be careful or do some benchmark.

There are many columns in tables, see https://dev.mysql.com/doc/refman/8.4/en/information-schema-tables-table.html
I believe we should consider including all placeholders at this time.

It may hurt performance, must be careful or do some benchmark.

Should be ok because these data are all associated with a single database connection. Actually there is no concurrent access at all. Using locks is just to bypass rust compiler check.

@evenyag @fengjiachun Please take a look.

@evenyag just updated, please check again


gatsby: Add subtitle to Measure detail page

Added the norm to the Measure detail page.
Decided to make it part of the content, for some of the measures, especially with longer norms, it looked super weird to have it just under the H1 above the content.
This is my first PR into this project, not sure if I missed anything or not, so I'm open to any criticism :))
Not sure if there is a design somewhere, I just rolled with what my eyes told me.

Design can be seen here: https://www.figma.com/file/9avjIglqc1VVc84zHIoLha/CovidPortal?node-id=1%3A155

Desktop components are not implemented yet. But this is good for now.

Good job! 🚀

Well done, this was probably the biggest trouble I noticed just after release.


Shouldn't EmployeeID references to ReportsTo?

EmployeeID referring to EmployeeID seems weird. I tried it works, but shouldn't the foreign key be set to ReportsTo column?
https://github.com/jpwhite3/northwind-SQLite3/blob/591cd3253c327b1eed7155c1fec57464565c0932/Northwind.Sqlite3.create.sql#L279
I checked the Microsoft SQL samples for this table and they also have it referred to ReportsTo. Have a look at microsoft/sql-server-samples.
Correct foreignkeyconstraint should be:
FOREIGN KEY ([ReportsTo]) REFERENCES [Employees] ([EmployeeID])
		ON DELETE NO ACTION ON UPDATE NO ACTION

Thanks for reporting the issue. Script has been updated.


chore: cleaned up old rimraf dependecy

:tada: This PR is included in version 1.1.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


DataTable sorting indicator appears in wrong TableColumnHead

Demo of problem:
https://svelte.dev/repl/3a27073d9b4f48f383b90c1dfd7395e2?version=3.46.3
Steps to reproduce:

load REPL
click on table header "Name"
Sorting indicator appear in column "Rule" instead of "Name"

Version tested: 0.57.1

Nice find – the reactivity of the headers prop is buggy


Push credentials don't work on cluster re-creation

What happened?
I'm currently trying to integrate my workflow into devspace, and am frequently creating/destroying clusters to experiment. I'm trying to put as much of the workload on the K8S cluster so that my warm laptop can survive the summer :-) OOT, but I have to say I love the potential devspace has. Thanks for the project!
If I re-create my Kubernetes cluster and container registry (via terraform destroy && terraform apply), I'm unable to build the image via kaniko, as it says it doesn't have permissions to push image:
[info]   Using namespace 'joaomlneto'
[info]   Using kube context 'admin@cluster'
[info]   Couldn't find a running docker daemon. Will fallback to kaniko
[info]   Building image 'rg.nl-ams.scw.cloud/my-namespace/myimage:XXXXXXX' with engine 'kaniko'
[warn]   Using CMD statement for injecting restart helper because ENTRYPOINT is missing in Dockerfile and `images.*.entrypoint` is also not configured
[done] √ Uploaded files to container
[done] √ Build pod has started
[info]   Printing logs of pod:container devspace-build-kaniko-pflrt:kaniko
E0424 21:53:27.458058       1 aws_credentials.go:77] while getting AWS credentials NoCredentialProviders: no valid providers in chain. Deprecated.
	For verbose messaging see aws.Config.CredentialsChainVerboseErrors
error checking push permissions -- make sure you entered the correct tag name, and that you are authenticated correctly, and try again: checking push permission for "rg.nl-ams.scw.cloud/my-namespace/myapp:XXXXXXX": creating push check transport for rg.nl-ams.scw.cloud failed: GET https://api.scaleway.com/registry-internal/v1/regions/nl-ams/tokens?scope=repository%3Amy-namespace%2Fmyapp%3Apush%2Cpull&service=registry: unexpected status code 401 Unauthorized: {"message":"Unauthorized request"}
[fatal]  error building image rg.nl-ams.scw.cloud/my-namespace/myapp:XXXXXXX: error building image (Exit Code 1)

Performing docker login [host] -u [user] -p [password] results in Login Succeeded.
I've checked the token used by kaniko matches that of my docker client.

Refreshing my credentials makes it work again (until re-creation). I'm unsure if this is related to the cloud provider, or to devspace.
As an alternate workaround I've tried to delete the .devspace folder and re-run devspace dev, but it doesn't work.
How can we reproduce the bug?

Create cluster and container registry with Scaleway
Use kaniko to build image and check everything works properly
Destroy and re-create the cluster (and edit ~/.kube/config with the new configuration)
Attempt to create the image using kaniko -- won't work due to insufficient push permissions.
Verify that credentials used by kaniko match those of the local docker client, and that docker login succeeds.
Create/refresh security and perform docker login with the new credentials.

Local Environment:

DevSpace Version: devspace version 5.12.1
Operating System: mac
Deployment method: helm (component chart)

Kubernetes Cluster:

Cloud Provider: Scaleway
Kubernetes Version:

Client Version: version.Info{Major:"1", Minor:"21", GitVersion:"v1.21.0", GitCommit:"cb303e613a121a29364f75cc67d3d580833a7479", GitTreeState:"clean", BuildDate:"2021-04-08T21:16:14Z", GoVersion:"go1.16.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.6", GitCommit:"8a62859e515889f07e3e3be6a1080413f17cf2c3", GitTreeState:"clean", BuildDate:"2021-04-15T03:19:55Z", GoVersion:"go1.15.10", Compiler:"gc", Platform:"linux/amd64"}

Anything else we need to know?
I'm unsure if this is caused by some stale devspace configuration lingering, or a problem with the cloud provider. I am more inclined towards the former, but as I'm unsure I'd ask you for assistance for debugging :-)

/kind bug

@joaomlneto thanks for creating this issue! For me it look like the scaleway provider is the issue and that the token (that is used by devspace) is invalid after you recreate the cluster, until you relogin with your username and password which refreshes the token.
Internally, DevSpace does not cache any registry credentials and will always just read the token from docker, but if docker returns a no longer valid token, there is not much we can do unfortunately.

Indeed. Problem seems to be with docker caching the token from the previous registry which stops being valid on registry recreation.
Thanks for the clarification!


Problem with restoration of non-english messages

Hi and congrats for your very helpful app. I have used the app in the past and worked perfectly.
 I checked the messages on gmail and they showed up nicely. Then I tried to restore the messages on my new LG G3. The restoration was successfull but I noticed that non-english charachters (actually greek) characters do not appear correctly. When I click view original message in gmail, I notice that the sms content is exactly the same with what appears on my phone. It seems that while the browser (chrome) does the conversion correctly, the messaging app doesn't. It is definetely an encoding issue. I found that any greek character that does not have an english equivalent is represented with its utf8 code when performing the backup. While chrome does the translation nicely when showing the message, the messaging app does not. Any suggestions on how to fix this issue. I tried changing the encoding in the messages app but it didn't work. I also tried using hangouts to display the message but the issue is still there. Is the fact that not all characters are transformed to unicode when doing the backup the problem?

duplicate of #519

Thanks for the immediate reply! Works perfectly again!


Don't count users when CMS_RAW_ID_USERS=True

Description
When using CMS_RAW_ID_USERS=True on a Postgres database with many users, counting the users is slow and will always yield the same result.
Only count users when using an integer value as a threshold and reuse the same logic for both PagePermissionInlineAdmin and
GlobalPagePermissionAdmin.
Background
When looking into performance issues for one of our websites, we found that Django CMS would regularly count all users in our Postgres database. Unfortunately, SELECT COUNT(*) FROM huge_table is slow in Postgres and easily takes 500ms or more if you have more than a million records in your table (exact numbers depend on the DB host of course).
I first looked into improving the speed of the count query, but this is not easy to achieve in a way that is compatible with different DB backends.
The easiest option was to not run this query at all when using the documented and widely used (AFAICT) CMS_RAW_ID_USERS = True.
The change should be backward compatible with the following exceptions:

CMS_RAW_ID_USERS = True will use raw IDs, even if you have just one or zero users in the DB (since 2 > True but not 1 >  True in Python). Since the current behavior could be considered a (rather academic) bug, I don't think this is a deal-breaker.
CMS_RAW_ID_USERS = 0 will trigger a count and use raw IDs when there's one user or more. If that's a problem, I could check if threshold is "truthy" like in the original implementation.

Checklist

[x] I have opened this pull request against develop
[x] I have updated the CHANGELOG.rst
[x] I have added or modified the tests when changing logic

Hi @vinitkumar, it seems my earlier rebase ran into the colors/faker issue on the CI system. I've just rebased again - hoping this now runs through.

Merged through #7414


Set service account for executors

To use custom/private Docker images for Spark drivers and executors, one needs to specify an image pull secret. The consensus in https://github.com/apache-spark-on-k8s/spark/issues/42 was to set the image pull secret onto the default service account in the namespace, and implicitly use that. However, in the case of a cluster that's using RBAC as per this Spark on K8s documentation, Spark will be using a non-default service account, and the image pull secret should be on that service account rather than a global namespace service account.
The difficulty is that, as far as I can tell, there's no way to tell Spark on K8s to spin up executors to use that custom service account, even though there is a way to set the driver's service account with spark.kubernetes.authenticate.driver.serviceAccountName. So the ask in this ticket is to add a comparable mechanism for setting the service account to use for the executors, so that the executors can use the same RBAC configuration and image pull secrets as the driver.

So sounds like we should extend the scope of the configuration property to cover both the driver and executors, e.g., something like spark.kubernetes.authenticate.serviceAccountName.

That would work great. Or separate settings for each of driver and executor. Either approach would satisfy this use case.


check hasOwnProperty when looping over modules

Using the DedupePlugin together with the core-js polyfill caused IE8 to fail with the message "Object doesn't support this property or method", because the for...in includes properties from the prototype chain of the polyfilled Array object. Using a regular for loop fixes this.

I've amended the commit to use hasOwnProperty instead

Amended to use Object.prototype.hasOwnProperty directly

Thanks


Added a featured gallery/carousel

Adds a featured sliding carousel to the QML page. Key features include:

Uses lightslider for the sliding, responsive, touch-friendly carousel.

Currently contains three demos, as cards: data re-uploading, doubly stochastic, and barren plateaus.

Slides one at a time.

On desktops, two cards are visible at once; on mobile, one is visible at a time.

Description is a (very) condensed summary.

Click the card to go to the demo.

@co9olguy, implemented the chevron buttons rather than the indicators :slightly_smiling_face:


Set Color Camera as default Pose Source for AR Foundation Tracked Pose Driver

For the Tracked Pose Driver it would be nice if the Pose Source defaulted to Color Camera.

Never mind, looks like it gets set in the camera profile


Gradient clipping

Add optional gradient clipping to Q-learning algorithms.
Update tests to test algorithms with gradient clipping.

https://github.com/takuseno/d3rlpy/pull/427

Add both gradient clipping and lr scheduler.


Embedded plugin window has a wrong position

Copied from issue #23 discussion:
I can load the plugin and I can hear it's working but the gui is black (which is new so there is evidence that something has changed):

When I maximize the window I see a part of the GUI (but I cannot click anything because probably the real 'hitboxes' are somwhere else in one of our 11 dimensions):

As soon as I close the plugin window Ardour freezes and crashes. Tested with LE456 64bit, LE456 32 bit and Amplitube 4 64 bit.
I did update my links using the Airwave manager.
I'm on Ubuntu 14.04.4 LTS (kernel 3.13.0-83-generic, x86_64)
I have Ardour 4.7.0, jack2 (1.9.11-20160223) started using Cadence with the following settings:

wine --version produces 'wine 1.8', but synapic shows it better I think:

Fyi, in release 1.3.1 I couldn't control the GUI, but in 1.3.2 I can, so I probably made a faulty assumption that it was related to the offset.

It seems that you're using wine with RT patches (wine-rt). I don't know whether it's the reason or not, but I think that it could be the root of the problem. At least, I highly recommend you to try plain wine or wine-staging (especially since you have it already installed). The RT patchset gives you almost no benefits. You could have a lower latency with it but a plain wine could also give you a relativetly low (256 frames) latency. It's good enough for producing music, as I think.

Ok, removed RT and I'm now at 1.6.2. GUI is still shifted, but Ardour does not crash anymore when closing the embedded window.

Could this be related to my dual monitor setup? I have a second monitor on the left of my primary screen but my X,Y coordinates are actually 0 at my left (secondary screen). So if I start the embedded window om my secondary screen and then stretch it over to my primary I can see the complete GUI:

I don't think that this should matter because the X,Y zero's should be in the embedded window itself, but you never know.

wine 1.6 is too old. Could you try wine 1.8?
sudo add-apt-repository ppa:ubuntu-wine/ppa
sudo apt-get update
sudo apt-get install wine

Of course, it could be an airwave issue. But I never saw this before and I know that XEMBED support was added to wine in branch 1.7.

Still the same:

Ok, then I will investigate this issue more presisely. I can't reproduce it on my side, so it will not be easy :-/

I understand, just let me know what I can do to help testing this.
I just changed the resolution of my left screen to 800x600 to see if that changes the offset in the plugin window and it actually does:

So I can now see it on my primary screen and it has a left offset of 800 + edge of window + additional small black offset around the gui:

Hope this helps.

The offset could depend on the initial position of the outer window. But as I can see, only X coordinate has the offset, which is a little strange.

Indeed it's strange that it is only on the X axis, especially when you look at my screen layout which has a Y-offset already:

Do you mean the ardour window (as it's parent) as outer window? When I test it with ardour on my secondary screen before opening the plugin window it gives exactly the same result as viewed in the screenshots in the above posts.
When I remove my secondary screen the offset-issue is gone:

(The crashing issue still remains but that's possibly an unrelated bug)

I meant the window with buttons "add", "save", "delete", "bypass". Ardour creates it and then asks a plugin to place its editor inside as a child.
According to your description, it can be a bug in wine's implementation of the XEMBED protocol. I will test it tomorrow on my other computer, which has two screens.

I'm not skilled enough to understand your code (I just know a bit of asm and C), but I spot a difference in parameters used with the SetWindowPos between Ardour windows version and Airwave.
Airwave: SetWindowPos(hwnd_, 0, **0, 0**, rect->right - rect->left, rect->bottom - rect->top, 				SWP_NOACTIVATE | SWP_NOMOVE);
Ardour (windows version): SetWindowPos (fst->windows_window, 0, **9999, 9999**, 2, 2, 0);
Sorry if this is bollocks... a bit of wild guessing here. I should actually test this myself before posting but because of my lack of skills I'm unable to compile Airwave myself.
Btw, thanks a lot already on your effort concerning this issue! Greatly appreciated.

I can't reproduce your behaviour on my two-screen configuration, it's very strange. I still tend to think that your wine could be too old.

I can't explain the values of 9999 for the SetWindowPos call too. But this Ardour's code is related to FST (FeSTige) and it's not executed when you're using the airwave.

it's all in pieces. Linux VST GUI size request in Ardour is in gtk2_ardour/lxvst_plugin_ui.cc:81
    _socket.set_size_request(
        new_width + _vst->state()->hoffset,
        new_height + _vst->state()->voffset)

new_width, new_height are from the plugin's "audioMasterSizeWindow" callback (libs/ardour/session_vst.cc:326,  the offsets are zero in case of LXVST (calloc() in vstfx_new ./libs/ardour/linux_vst_support.cc:85`)
Unless I've missed something and short of a hidden bug somewhere, the only explanation would be that the VST plugin returns a wrong size in audioMasterSizeWindow.

PS. there was a change in Ardour 4.4-643-gd959461  to allow resizable/scalable Linux VST UIs (mainly for the benefit of u-he).   Versions prior to 4.4 did not forward size-requests to X11 child windows but Ardour 4.7 mentioned above does.

Thanks for checking @x42!
Sorry for the late reply @phantom-code. I'm pretty sure I have wine 1.8. Additional wine related installed packages: wine-gecko, playonlinux, wine-mono, wineasio.

I have the same problem in Ardour, and also in Renoise, but not in Bitwig.
Wine version is a staging 1.9.9 patched with the xembed patch.

@Llamatron2112 what distro are you using? Do you also have a dual monitor setup? If so on which side is your primary monitor?

I am using Arch, with a dual monitor setup and my primary monitor is to the
right. Configuring my display to declare the secondary monitor to the right
"fixes" the issue.
2016-06-04 17:21 GMT+02:00 Snevzor<EMAIL_ADDRESS>
@Llamatron2112 https://github.com/Llamatron2112 what distro are you
using? Do you also have a dual monitor setup? If so on which side is your
primary monitor?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/phantom-code/airwave/issues/41#issuecomment-223761268,
or mute the thread
https://github.com/notifications/unsubscribe/ACngsPN0uLMN0NtfuC6dII1LaoxuY-asks5qIZgHgaJpZM4H8-8j
.

Ok so it's not desktop environment dependent. Still no clue if this bug originates in Wine, Ardour or Airwave :(

Im facing same problem with xfce4 and dual screen setup.
if I disable dual screen I can see plugin correctly, but with 2 monitors positioning is wrong.
I also get a wine error on closing ardour, but saving just befor makes it just annoying, not a stopper for me.

It looks like a wine or xfce issue. Sadly, I barely can do anything here.

After finally stumbling here after many hours of troubleshooting...
My Setup:

openSUSE Tumbleweed with KDE Desktop Environment
Reaper 5.413 (native Linux version)
Latest airwave from git.

I was elated to get Reaper working with some old school Windows based VST plugins (Guitar Rig 3 for an old project). This was on my laptop. When I moved to an identical setup using my desktop machine (dual monitors) and my plugins were blank.
Sure enough, going back to a single monitor fixes the issues and plugins render as expected.

@phantom-code Work around for this issue: run the following command:
# Disable primary monitor before launching DAW software
xrandr --noprimary

# Re-enable primary monitor after DAW session
xrandr --output <monitor_name> --primary

In my case I run xrandr --output DVI-D-0 --primary after closing Reaper. Of course, you can use the "Display Settings" configuration GUI in [insert desktop environment here] to set the same values.


Image upload ignored

Outline 📝
details unknown...
Why do you need to do it?
I'm having trouble uploading images. 🥺

Maybe the problem occurs when there is both a link card and an image? 🤔


Storage bit-for-bit comparison

@tomponline all green for the tests/storage-vm test against all snap channels. The execution time is ~30s longer than before.


SFI - ROPC: First group of updates

Contributes to #41954
(Hide whitespace changes.)

Added an include file with the recommendation to use managed identities where possible.
Other cleanup according to Learn Linter.
Replace .NET Framework 4.6.1 name include file.
Remove unnecessary explicit bookmarks.
Format stuff as code.
Format tables.

Internal previews
Toggle expand/collapse

📄 File
🔗 Preview link

docs/core/additional-tools/self-signed-certificates-guide.md
docs/core/additional-tools/self-signed-certificates-guide

docs/core/porting/modernize.md
docs/core/porting/modernize

docs/framework/configure-apps/file-schema/wcf/add-of-commonparameters.md
docs/framework/configure-apps/file-schema/wcf/add-of-commonparameters

docs/framework/configure-apps/file-schema/wcf/commonparameters.md
docs/framework/configure-apps/file-schema/wcf/commonparameters

docs/framework/configure-apps/file-schema/wcf/serviceauthorization-element.md
docs/framework/configure-apps/file-schema/wcf/serviceauthorization-element

docs/framework/configure-apps/file-schema/wcf/workflowruntime.md
docs/framework/configure-apps/file-schema/wcf/workflowruntime

docs/framework/configure-apps/file-schema/windows-workflow-foundation/behavior-of-servicebehaviors-of-workflow.md
docs/framework/configure-apps/file-schema/windows-workflow-foundation/behavior-of-servicebehaviors-of-workflow

docs/framework/configure-apps/file-schema/windows-workflow-foundation/sqlworkflowinstancestore.md
docs/framework/configure-apps/file-schema/windows-workflow-foundation/sqlworkflowinstancestore

docs/framework/configure-apps/file-schema/windows-workflow-foundation/system-servicemodel-of-workflow.md
docs/framework/configure-apps/file-schema/windows-workflow-foundation/system-servicemodel-of-workflow

docs/framework/windows-workflow-foundation/configuring-tracking-for-a-workflow.md
docs/framework/windows-workflow-foundation/configuring-tracking-for-a-workflow

docs/framework/windows-workflow-foundation/csharp-expressions.md
docs/framework/windows-workflow-foundation/csharp-expressions

docs/framework/windows-workflow-foundation/dynamic-update.md
docs/framework/windows-workflow-foundation/dynamic-update

docs/framework/windows-workflow-foundation/exceptions.md
docs/framework/windows-workflow-foundation/exceptions

docs/framework/windows-workflow-foundation/how-to-deserialize-instance-data-properties.md
docs/framework/windows-workflow-foundation/how-to-deserialize-instance-data-properties

docs/framework/windows-workflow-foundation/how-to-enable-persistence-for-workflows-and-workflow-services.md
docs/framework/windows-workflow-foundation/how-to-enable-persistence-for-workflows-and-workflow-services

docs/framework/windows-workflow-foundation/how-to-enable-sql-persistence-for-workflows-and-workflow-services.md
docs/framework/windows-workflow-foundation/how-to-enable-sql-persistence-for-workflows-and-workflow-services

docs/framework/windows-workflow-foundation/how-to-update-the-definition-of-a-running-workflow-instance.md
docs/framework/windows-workflow-foundation/how-to-update-the-definition-of-a-running-workflow-instance

docs/framework/windows-workflow-foundation/net-framework-3-0-wf-in-net-framework-4-interop.md
docs/framework/windows-workflow-foundation/net-framework-3-0-wf-in-net-framework-4-interop

docs/framework/windows-workflow-foundation/pausing-and-resuming-a-workflow.md
docs/framework/windows-workflow-foundation/pausing-and-resuming-a-workflow

docs/framework/windows-workflow-foundation/pick-activity.md
docs/framework/windows-workflow-foundation/pick-activity

docs/framework/windows-workflow-foundation/primitives-activities-in-wf.md
docs/framework/windows-workflow-foundation/primitives-activities-in-wf

docs/framework/windows-workflow-foundation/required-arguments-and-overload-groups.md
docs/framework/windows-workflow-foundation/required-arguments-and-overload-groups

docs/framework/windows-workflow-foundation/runtime-activities-in-wf.md
docs/framework/windows-workflow-foundation/runtime-activities-in-wf

docs/framework/windows-workflow-foundation/samples/database-access-activities.md
docs/framework/windows-workflow-foundation/samples/database-access-activities

docs/framework/windows-workflow-foundation/sql-workflow-instance-store.md
docs/framework/windows-workflow-foundation/sql-workflow-instance-store

docs/framework/windows-workflow-foundation/store-extensibility.md
docs/framework/windows-workflow-foundation/store-extensibility

docs/framework/windows-workflow-foundation/tracking-events-reference.md
docs/framework/windows-workflow-foundation/tracking-events-reference

docs/framework/windows-workflow-foundation/tracking-participants.md
docs/framework/windows-workflow-foundation/tracking-participants

docs/framework/windows-workflow-foundation/variable-and-argument-tracking.md
docs/framework/windows-workflow-foundation/variable-and-argument-tracking

docs/framework/windows-workflow-foundation/workflow-tracking-and-tracing.md
docs/framework/windows-workflow-foundation/workflow-tracking-and-tracing

[!NOTE]
This table shows preview links for the 30 files with the most changes. For preview links for other files in this PR, select OpenPublishing.Build Details within checks.

Thanks @BillWagner!


Check for changes before save prompt

We now compare the Json files to see if something has changed before asking the player to save. It's very brute force and can probably be optimized. But it works good enough.
fixes #358

SonarQube analysis reported 5 issues

 5 minor

Watch the comments in this conversation to review them.
4 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AncientFungus:LGEPR: Cppcheck cannot find all the include files (use --check-config for details)
 SceneEditor.cpp#L20: Member variable 'SceneEditor::sceneSelector' is not initialized in the constructor.
 Resources.hpp#L90: Unused private function: 'ResourceList::GetInstance'
 Hymn.cpp#L171: Variable 'model' is assigned a value that is never used.


Fix go releaser githbub action.

Fixes #165
This action failed for the last release.
Updating based on the github marketplace docs

Thank you for working on this!


metrics stream handler doesn't understand error_response command

When cowboy env has 2 stream handlers configured ([cowboy_metrics_h, cowboy_stream_h]) and unexpected error happens in one of request handlers, cowboy_http receives an 'EXIT' message, which is translated into the following chain of commands by cowboy_stream_h stream handler (example):
[{error_response,500,#{<<"content-length">> => <<"0">>},<<>>},
 {log,error,
     "Ranch listener ~p, connection process ~p, stream ~p had its request process ~p exit with reason ~999999p and stacktrace ~999999p~n",
     [bower_app,<0.366.0>,1,<0.367.0>,
      {badmatch,2},
      [<STACKTRACE>]]},
 {internal_error,
     {'EXIT',<0.367.0>,
         {{badmatch,2},
          [<STACKTRACE>]}},
     'Stream process crashed.'}]

cowboy_metrics_h does not understand error_respone and does not update state with the response information as it does for response command.
This results in undefined information about response on terminate step.

I'm afraid you're right. Nice find, thanks! I'll see if I can fix it next week.

Fixed locally, push soon.


implements #252 by allowing to add freeform headers to the client.

ready to merge @duemir ?

I am afraid I have not done much testing on this one @aheritier
I am also not sure that Codeship build ran the integration tests

no I think it's still not configured to do it :(
Arnaud Héritier
Customer Engagement, Support Delivery Manager
CloudBees, Inc.
[image: CloudBees-Logo.png] http://www.cloudbees.com
Mobile: +33 6 89 76 64 24
Skype / Twitter / GitHub: aheritier
On Thu, Apr 4, 2019 at 3:59 AM Denys Digtiar<EMAIL_ADDRESS>wrote:

I am afraid I have not done much testing on this one @aheritier
https://github.com/aheritier
I am also not sure that Codeship build ran the integration tests
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cloudbees/zendesk-java-client/pull/258#issuecomment-479720789,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKqCLP2c72KIgJZr1vyC2MweeKvC4_Zks5vdVxsgaJpZM4U3Xum
.


Pod & Packed traits

I'm proposing adding Pod and Packed traits and a write_initialized intrinsic to make it easier to work with structured low-level APIs.
Previous discussions

https://github.com/rust-lang/rfcs/pull/1387
https://github.com/rust-lang/rfcs/pull/936

A sized type T is Pod if any [u8; sizeof(T)] can safely be transmuted to T.
A (possibly unsized) object is Pod if any byte in its object representation can safely be overwritten by any other byte.
All integer types are Pod. All raw pointers to sized types are Pod. Arrays of Pod types are Pod. Tuples, structs and unions which contain only Pod types and which do not have invariants are Pod.

A sized type T is Packed if any object of type T can safely be transmuted to [u8; sizeof(T)].
A (possibly unsized) object is Pod if any byte in its object representation can safely be read as an u8.
All integer types are Packed. All raw pointers on currently supported platforms are Packed. Arrays of of Packed types are Packed. Whether tuples, structs and unions are Packed depends on their contents.
A Packed trait cannot be implemented safely outside the compiler.

Pod and Packed traits make it possible to write safe APIs that are useful in systems programming. For example:
fn pod_zeroed<T: Pod>() -> T;

// Err if sizeof(U) != sizeof(T)
fn pod_read<T: Pod, U: Packed + ?Sized>(u: &U) -> Result<T>;

// Err if sizeof(U) < sizeof(T)
fn pod_read_init<T: Pod, U: Packed + ?Sized>(u: &U) -> Result<T>;

// Err if sizeof(U) is not a multiple of sizeof(T)
fn pod_iter<T: Pod, U: Packed + ?Sized>(u: &U) -> Result<impl Iterator<Item = T>>;

// Err if sizeof(U) != sizeof(T)
fn pod_write<T: Pod + ?Sized, U: Packed + ?Sized>(u: &U, t: &mut MaybeUninit<T>) -> Result<()>;

Pod and Packed do not cover all use cases that deal with transmuting between bytes and objects. For example, it would be useful if the compiler provided the following intrinsic:
/// Overwrites `t` with `u`, replacing all padding bits of `U` by an unspecified bit pattern.
/// Err if sizeof(u) != sizeof(T).
fn write_initialized<T: Pod + ?Sized, U: ?Sized>(u: &U, t: &mut MaybeUninit<T>) -> Result<()>;

This would cover the usecase of writing data structures with padding bytes into memory.

Both Packed and Pod would not be auto-traits. Like Copy, every type has to opt into being Pod or Packed. Like Copy the compiler would verify at compile time that

types implementing Pod only contain other Pod types recursively
types implementing Packed only contain other Packed types and do not have padding bits

I assume that Packed would be more controversial that Pod due to its platform dependence.

You may want to take a look at the safe transmute working group.

A (possibly unsized) object is Pod if any byte in its object representation can safely be read as an u8.
I guess you mean Packed here?

Yes


Add ldap login function

Problem Summary:
What is changed and how it works?
Proposal: xxx
Feel free to ping committers for the review!
Contribution Checklist

Make sure that the pull request corresponds to a GITHUB issue.

Name the pull request in the form "[Feature] [component] Title of the pull request", where Feature can be replaced by Hotfix, Bug, etc.

Minor fixes should be named following this pattern: [hotfix] [docs] Fix typo in README.md doc.

-->
Purpose of this pull request

please rebase your pr. thx


ASAN support for Linux and Mac

Test by running ./build_dependencies.sh --release --asan llvm-10.
You'll also probably want to compile your tool with something like the following:
CXXFLAGS="-fsanitize=address -fno-omit-frame-pointer -fno-optimize-sibling-calls -ffunction-sections -fdata-sections -Wl,-undefined,dynamic_lookup" \
  CCFLAGS="-fsanitize=address -fno-omit-frame-pointer -fno-optimize-sibling-calls -ffunction-sections -fdata-sections -Wl,-undefined,dynamic_lookup" \
  LDFLAGS="-fsanitize=address -Wl,-undefined,dynamic_lookup" \
  cmake -G Ninja -DCMAKE_INSTALL_PREFIX=$(pwd)/install \
  -DVCPKG_ROOT="<path_to_vcpkg>" \
  -DVCPKG_TARGET_TRIPLET=x64-linux-rel-asan \
  ..

Updated build script to more easily compile with ASAN --asan

Workaround for failing build in LLVM with manually added extra compilation flags. Now just looks for the VCPKG_USE_SANITIZER variable

Brought in gtest to add a patch (ports/gtest/remove-werror.patch) to remove -Werror since there are some warnings with the added sanitizer flags.

[x]  Added new triplet with command line flags for ASAN. These command-line flags might be better used in a toolchain file. See here for example https://github.com/Neumann-A/my-vcpkg-triplets. EDIT: After further investigation, using a CMake toolchain with CMAKE_*_FLAGS, etc. for the sanitizer options did not work for non-cmake project XED, but it did work for CMake project GTest. Might need to wait for upstream to better support forwarding these CMAKE variables to non-cmake projects.

[x] Add osx asan support

[x] Fix build_dependencies script because I think the argparsing won't work if --asan is before --release

Are there a better flags for Address sanitizer support? I took these from a different project


Fixed undefined reference to onAllFinished callback in _requestUrl

I found an undefined reference to the onAllFinished callback within _requestUrl.
In the shouldSkip callback of workExecutor.submit, if the url should be skipped then _finishedCrawling is passed onAllFinished, which does not exist in this context.
Passed callback to _finishedCrawling instead.

Hi, thanks, good catch, onAllFinished is indeed undefined in this context, however it should not be callback that should be passed to there. It still should be onAllFinished. I will merge the PR and make the necessary changes.


Extra newline character in the output of changie latest

Description
The output of changie latest contains an extra newline character. This is not the case for other commands, such as change next patch.
Reproduction Steps
Steps to reproduce the behavior:

Use changie latest with the CLI or the official GitHub Action

Check the output includes an extra newline:
$ changie latest | cat -e
v1.10.1$

This is not the case of next:
$ changie next patch | cat -e
v1.10.2

What happened
Using the output of the GitHub action in a filename, leads to an ENOENT error:
Error: ENOENT: no such file or directory, open '.changes/0.4.2
.md'

Expected behavior
The output doesn't have any extra newlines.
I'm working on a PR if this indeed a bug.

Good find, those invisible characters are hard to track down for sure.


feature: Non-deferred Function calls

The current Function::Call is soon to be deprecated and directs embedders to the new overload that takes a Context (usually retrieved from the current isolate?) and returns a MaybeLocal.
Nan currently provides a way to make deferred calls to JS functions in a V8/node version independent manner (through Nan::Callback which uses node::MakeCallback).
It would be nice for Nan to have a primitive for making non-deferred js function calls for when the original Call overload is finally deprecated and removed.

See the discussion that I improperly started in #284

The old Function::Call is already removed. #496

@kkoopa Ah cool. Thanks for that!
Hope starting that conversation over there didn't cause too much confusion :/ kind of new in this neck of the woods.

No problem. However, you'll want to patch your own code until this is released, which will be when there are enough new features to warrant a new release, maybe in a month or two.
On October 15, 2015 7:34:19 PM GMT+03:00, Joshua Perry<EMAIL_ADDRESS>wrote:

@kkoopa Ah cool. Thanks for that!
Hope starting that conversation over there didn't cause too much
confusion :/ kind of new in this neck of the woods.

Reply to this email directly or view it on GitHub:
https://github.com/nodejs/nan/issues/495#issuecomment-148447346

Not a prob. Was just hoping to make the future better for everyone!
Thank you again for your help.

Good, thanks for reporting this. I had completely missed it.
On October 15, 2015 7:42:11 PM GMT+03:00, Joshua Perry<EMAIL_ADDRESS>wrote:

Not a prob. Was just hoping to make the future better for everyone!
Thank you again for your help.

Reply to this email directly or view it on GitHub:
https://github.com/nodejs/nan/issues/495#issuecomment-148450013


add more options to clean unused strings only, ignore drawables or ig…

…nore dimensions

no tests + breaks existing tests

Hi @ialmetwally
do you have a interest on making sure your changes are tested and don't break our existing tests? Would love to merge it, but we need that do move forward. If not I'd just close this PR.

@philippb @ca77y I've fixed the tests and modified the script accordingly, feel free to merge if everything is ok

@ca77y please look at this and decide your self.

@ialmetwally thanks

@ca77y can we make sure we merge this in


Map DotNetRestoreSources => RestoreSources for metapackage builds

I did quick validation. The error message before says:
:45]C:\b\w\33bdfc1cae7b2a38\.w\Microsoft.AspNetCore.App\Microsoft.AspNetCore.App.csproj : error NU1102: Unable to find package Microsoft.AspNetCore.NodeServices with version (= 2.1.0-preview2-30312)
[11:41:45]C:\b\w\33bdfc1cae7b2a38\.w\Microsoft.AspNetCore.App\Microsoft.AspNetCore.App.csproj : error NU1102:   - Found 33 version(s) in nuget.org [ Nearest version: 2.1.0-preview1-final ]

With this change it shows that the additional restore sources are available.

Sounds good to me.


Exclude subfolders?

Is there a way to exclude subfolders from a list so that the results display only those pages that are in a root folder?
I've tried
list from "Recipes"
where !contains(file.path, "Recipes/")

The forward slash is not understood as part of the exclusion until a letter is added after it,  e.g., "Recipes/A", "Recipes/B", etc.  which means each subfolder would need to be explicitly excluded.

Hmm, I should look into an easier way of doing it, but you can achieve this with regexmatch:
LIST FROM "Recipes"
WHERE regexmatch("Recipes/[^/]+.md", file.path)

This effectively discards any paths which have another '/' in them (i.e., in a subfolder).

I'd like to add to this, along with excluding subfolders, I feel it'd be important (and useful) to be able to indent the subfolders' contents if you don't want to exclude them.
This would enable you to build a full, proper index.

@firinael can you explain what you mean by "indenting contents"?

Closing as duplicate of #66


Hash join

Add hash join for google spanner, furhter documentation can be found here.
https://cloud.google.com/spanner/docs/query-execution-operators#hash-join

Coverage increased (+0.003%) to 98.195% when pulling 87a53d82a20ad9f9ab04d6ace297e67c18ea6d7a on wei-ancoris:hash_join into 0d1c4048baa14226364e45838724cd2781839086 on kayak:master.


re structure

Overview
Changes

Fixes: # .

/azp run mrtk_docs

Waiting on docs run, changes seem okay.

Can you fix the doc build breaks please:
Build failed.
[19-04-24 10:10:29.536]Warning:BuildCommand.BuildCore.Build Document.CompilePhaseHandlerWithIncremental.TocDocumentProcessor.Prebuild.BuildTocDocumentUnable to find either toc.yml or toc.md inside Documentation/. Make sure the file is included in config file docfx.json!
[19-04-24 10:10:29.536]Error:BuildCommand.BuildCore.Build Document.Load.TocDocumentProcessorD:\a\1\s\Documentation\toc.yml is not a valid TOC File: D:\a\1\s\Documentation\toc.yml is not a valid TOC file, detail: (Line: 11, Col: 5, Idx: 212) - (Line: 11, Col: 5, Idx: 212): While parsing a block collection, did not find expected '-' indicator..
[19-04-24 10:10:29.536]Error:BuildCommand.BuildCore.Build Document.Load.TocDocumentProcessorUnable to load file 'Documentation/toc.yml' via processor 'TocDocumentProcessor': D:\a\1\s\Documentation\toc.yml is not a valid TOC File: D:\a\1\s\Documentation\toc.yml is not a valid TOC file, detail: (Line: 11, Col: 5, Idx: 212) - (Line: 11, Col: 5, Idx: 212): While parsing a block collection, did not find expected '-' indicator..
1 Warning(s)

/azp run mrtk_docs

@Yoyozilla  Sorry, but what is this restructuring exactly? Is it the table of contents for github.io page? Could you provide a little bit more detail in the pull request please?

This is in response to some of the feedback we have gotten.
The documents today are more like feature description and overview vs how-to.
How-to should be scenario/task focused to teach people how to complete a tast/scenario.
I am moving the doc ToC reflect more of this.

/azp run mrtk_docs

/azp run mrtk_docs

/azp run mrtk_docs

Hi yoyo, it looks like the docs build is still failing. The error is:
(Documentation/toc.yml#L0) Invalid file link:(~/Documentation/README.md).

I think also the general feedback was to have in the PR description what is being changed and why. (If you could take your response to Julia's feedback and paste it into the PR description that would be good)

Per our offline chat, I deleted the stale migration readme, but this causes a merge conflict in yours because I had to update the toc.yml
https://github.com/Microsoft/MixedRealityToolkit-Unity/pull/4090

/azp run mrtk_docs

@Yoyozilla ignore if you know / do this already: you can run docfx locally - it just takes a couple of seconds. that way you can easier fix issues / iterate on issues in this PR without having to wait for the CI builder all the time :)

/azp run mrtk_docs


add ISRUC dataset (#103)

ISRUC added

Added documentation

minor

fix download issue in isruc, add example and logs

oops, forget to update the doc

Checked in #103 , I will directly merge into master.


UTF-8 conversion is significantly slower with BSON than regular JSON

Hi there,
I just did some quick performance testing in order to test the performance of BSON against JSON (and kryo). I noticed something that may be relevant for you. In my (simplistic) test, BSON took 3 times as long to serialize the same workload as JSON (both with writeValueAsBytes). Upon closer inspection, it seems that the UTF-8 conversion of the field names is at fault.
Here's what JVisualVM has to say about JSON:

... and (same workload) for BSON:

Expanding the details on the BSON variant furhter always ends up at the same method: DynamicOutputBuffer.putUTF8(...) calling into java.nio.charset.CharsetEncoder.encode(...). That's where the time is lost in comparison to the blank JSON variant.
This is quite a significant difference and I thought I should at least report it.
Here's the full project which produced the above results (it's in Kotlin; just run the static main method of Main.kt)
BsonTest.zip
I might be wrong with my analysis, I was merely experimenting, so please take it with a grain of salt.

Hi @michel-kraemer,
thanks for the very quick reply. I just took your snapshot for a test drive, and the results are indeed much better than before. Here's what I'm getting on my machine with the benchmark I sent to you:
BENCHMARK: Serialize 1000000 Persons
| Name | Runs | Min (ms) | Max (ms) | Avg (ms) | StDev (ms) | Output Bytes |
----------------------------------------------------------------------------
| BSON |   10 |      468 |      634 |    501.2 |     59,405 |  6.9888961E7 |
| JSON |   10 |      311 |      325 |    315.5 |      4,225 |  7.0888962E7 |

BENCHMARK: Deserialize 1000000 Persons
| Name | Runs | Min (ms) | Max (ms) | Avg (ms) | StDev (ms) |
-------------------------------------------------------------
| BSON |   10 |     1409 |     1456 |   1438.2 |     17,163 |
| JSON |   10 |      867 |      903 |    881.3 |     10,120 |

BENCHMARK: Extract "name" from 1000000 Persons
| Name | Runs | Min (ms) | Max (ms) | Avg (ms) | StDev (ms) |
-------------------------------------------------------------
| BSON |   10 |     1148 |     1188 |   1173.1 |      9,864 |
| JSON |   10 |      581 |      617 |    595.6 |     11,289 |

So in the serialization case, it's a lot closer to vanilla JSON now. In the deserialization cases it's about 2x slower than vanilla JSON, as compared to the 4x slower than previously.
While I don't know very much about the charset encodings and arrangements of UTF-8 bits as such, a quick google search revealed this answer from stackoverflow which seems to be related. It basically states that the character encodings have "fast paths" which are optimized for certain inputs (mostly for bulk-encoding an array, rather than encoding individual characters). My guess is that vanilla JSON is just very good at leveraging those pre-optimized conversions. I've witnessed in the past that calling new String(bytes, Charsets.UTF_8) will often outperform any custom byte-array-to-string-conversion algorithm. To me it seems like the JDK (or standard library) does a lot of trickery under the hood to make this particular conversion fast, and you may fall through the holes in this performance safety net if you do too much custom coding in this area. Just my 5 cents, I'll stop rambling now :)
Thank you for your efforts, they're very much appreciated!


CodeIgniter's user guide page not found 404 error in first install

I just installed CI and saw the page welcome. This is good but.. when I clicked user guide then...

well... There is no directory named 'user_guide' actually but 'user_guide_src' is.

Did I something wrong??

I found a 'user_guide' directory in offical site (https://www.codeigniter.com/) but why isn't git?

Because you're supposed to use the final version, not the current git state.
Please note that we don't provide general support here. If you seek help with something, ask your questions on our forums instead.


(maint) Fix resource order for installing splunk forwarders

Prior to this commit, when applying the forwarder class to any agents
the initial run would always fail because the module would attempt to
start the splunk service before it existed. This commit fixes that by
adding a notify param to the provider that sets up that forwarder
service.
Before this commit:
May 17 21:41:28 localhost puppet-agent[9660]: Retrieving pluginfacts
May 17 21:41:28 localhost puppet-agent[9660]: Retrieving plugin
May 17 21:41:30 localhost puppet-agent[9660]: Loading facts
May 17 21:41:37 localhost puppet-agent[9660]: Caching catalog for samurai-dog
May 17 21:41:37 localhost puppet-agent[9660]: Applying configuration version '1463521294'
May 17 21:41:37 localhost puppet-agent[9660]: (/Stage[main]/Staging/File[/opt/staging]/ensure) created
May 17 21:41:38 localhost puppet-agent[9660]: (/Stage[main]/Splunk::Forwarder/Staging::File[DownloadActivityServlet]/File[/opt/staging/splunk]/ensure) created
May 17 21:41:40 localhost puppet-agent[9660]: (/Stage[main]/Splunk::Forwarder/Staging::File[DownloadActivityServlet]/Exec[/opt/staging/splunk/DownloadActivityServlet]/returns) executed successfully
May 17 21:41:41 localhost puppet-agent[9660]: (/Stage[main]/Splunk::Forwarder/Package[splunkforwarder]/ensure) created
May 17 21:41:41 localhost puppet-agent[9660]: Could not unmask splunk:
May 17 21:41:41 localhost puppet-agent[9660]: (/Stage[main]/Splunk::Virtual/Service[splunk]/ensure) change from stopped to running failed: Could not unmask splunk:
May 17 21:41:41 localhost puppet-agent[9660]: (/Stage[main]/Splunk::Forwarder/File[/opt/splunkforwarder/etc/system/local/inputs.conf]/ensure) created
May 17 21:41:41 localhost puppet-agent[9660]: (/Stage[main]/Splunk::Forwarder/File[/opt/splunkforwarder/etc/system/local/outputs.conf]/ensure) created
May 17 21:41:41 localhost puppet-agent[9660]: (/Stage[main]/Splunk::Forwarder/File[/opt/splunkforwarder/etc/system/local/web.conf]/ensure) created
May 17 21:41:41 localhost puppet-agent[9660]: (/Stage[main]/Splunk::Forwarder/Splunkforwarder_input[default_host]/ensure) created
May 17 21:41:41 localhost puppet-agent[9660]: (/Stage[main]/Splunk::Forwarder/Splunkforwarder_output[tcpout_defaultgroup]/ensure) created
May 17 21:41:41 localhost puppet-agent[9660]: (/Stage[main]/Splunk::Forwarder/Splunkforwarder_output[defaultgroup_server]/ensure) created
May 17 21:41:41 localhost puppet-agent[9660]: (/Stage[main]/Splunk::Forwarder/Splunkforwarder_web[forwarder_splunkd_port]/ensure) created
May 17 21:41:43 localhost puppet-agent[9660]: (/Stage[main]/Splunk::Platform::Posix/Exec[license_splunkforwarder]/returns) executed successfully
May 17 21:41:43 localhost systemd: Reloading.
May 17 21:41:43 localhost systemd: Configuration file /usr/lib/systemd/system/auditd.service is marked world-inaccessible. This has no effect as configuration data is accessible via APIs without restrictions. Proceeding anyway.
May 17 21:41:43 localhost systemd: Reloading.
May 17 21:41:43 localhost systemd: Configuration file /usr/lib/systemd/system/auditd.service is marked world-inaccessible. This has no effect as configuration data is accessible via APIs without restrictions. Proceeding anyway.
May 17 21:41:43 localhost puppet-agent[9660]: (/Stage[main]/Splunk::Platform::Posix/Exec[enable_splunkforwarder]/returns) executed successfully
May 17 21:41:43 localhost puppet-agent[9660]: (/Stage[main]/Profile::Splunkforwarder/Splunkforwarder_input[messages-sourcetype]/ensure) created
May 17 21:41:43 localhost puppet-agent[9660]: (Stage[main]) Unscheduling all events on Stage[main]
May 17 21:41:43 localhost puppet-agent[9660]: Applied catalog in 6.17 seconds

After this commit:
root@badger ~]# puppet agent -t --no-use_cached_catalog
Notice: Local environment: 'production' doesn't match server specified node environment 'splunkitup', switching agent to 'splunkitup'.
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Loading facts
Info: Caching catalog for badger
Info: Applying configuration version '1463528182'
Notice: /Stage[main]/Staging/File[/opt/staging]/ensure: created
Notice: /Stage[main]/Splunk::Forwarder/Staging::File[DownloadActivityServlet]/File[/opt/staging/splunk]/ensure: created
Notice: /Stage[main]/Splunk::Forwarder/Staging::File[DownloadActivityServlet]/Exec[/opt/staging/splunk/DownloadActivityServlet]/returns: executed successfully
Notice: /Stage[main]/Splunk::Forwarder/Package[splunkforwarder]/ensure: created
Notice: /Stage[main]/Splunk::Forwarder/File[/opt/splunkforwarder/etc/system/local/inputs.conf]/ensure: created
Notice: /Stage[main]/Splunk::Forwarder/File[/opt/splunkforwarder/etc/system/local/outputs.conf]/ensure: created
Notice: /Stage[main]/Splunk::Forwarder/File[/opt/splunkforwarder/etc/system/local/web.conf]/ensure: created
Notice: /Stage[main]/Splunk::Forwarder/Splunkforwarder_input[default_host]/ensure: created
Notice: /Stage[main]/Splunk::Forwarder/Splunkforwarder_output[tcpout_defaultgroup]/ensure: created
Notice: /Stage[main]/Splunk::Forwarder/Splunkforwarder_output[defaultgroup_server]/ensure: created
Notice: /Stage[main]/Splunk::Forwarder/Splunkforwarder_web[forwarder_splunkd_port]/ensure: created
Notice: /Stage[main]/Splunk::Platform::Posix/Exec[license_splunkforwarder]/returns: executed successfully
Notice: /Stage[main]/Splunk::Platform::Posix/Exec[enable_splunkforwarder]/returns: executed successfully
Info: /Stage[main]/Splunk::Platform::Posix/Exec[enable_splunkforwarder]: Scheduling refresh of Service[splunk]
Notice: /Stage[main]/Splunk::Virtual/Service[splunk]/ensure: ensure changed 'stopped' to 'running'
Info: /Stage[main]/Splunk::Virtual/Service[splunk]: Unscheduling refresh on Service[splunk]
Notice: /Stage[main]/Profile::Splunkforwarder/Splunkforwarder_input[messages-sourcetype]/ensure: created
Notice: Applied catalog in 7.70 seconds

We run into the same issue. Fix works as designed (for the forwarder)

Would love to see this merged :)

This simple simple, safe and sensible 👍


Plex sync

Changes made

Ported from #401 with some changes (easier than fixing old pr first).
Changes from old pr:

Also sync seasons/episodes.
Use includeGuids query parameter (that was buried in a forum post... why...) to get tmdbId so we can directly get a show/movie without needing to rely on search.
Don't move activity modal component.
Also some other small changes, but not worth mentioning because I forgot.

Closes #359

dumbo plex auth token doesnt work against the local server for api requests when using another account :((((((((((((((
but it does work if i login via web ui and steal the auth token it generates which makes me think something is wrong in the oauth flow and we aren't asking for a token with permissions for our local server??? plos help no i dont need help ill figure it out tomorrow but also help

Figured it out, but in the progress moved plex auth tokens to a new user_services table because we now have two separate tokens (one for use against plex.tv and another for our home server).
At least we now have a starting point when integrating future services/integrations.
Jellyfin can stay how it is for now (and forever probably) to avoid breaking changes.


Encode link text in linkify core rule to avoid homograph attack

As mentioned in https://en.wikipedia.org/wiki/IDN_homograph_attack

The internationalized domain name (IDN) homograph attack is a way a malicious party may deceive computer users about what remote system they are communicating with, by exploiting the fact that many different characters look alike (i.e., they are homographs, hence the term for the attack, although technically homoglyph is the more accurate term for different characters that look alike).

Eg.  In the link "http://ebаy.com", "a" looks like Latin "a" but actually is the Cyrillic character. So if we linkify it, text will be "http://ebаy.com" but link will be "http://ebаy.com/"
After this change above text http://ebаy.com will be rendered as "http://ebаy.com/". Such that href and text values of anchor tag will be same.

This PR is buggy for 2 reasons:

It tries to use wrong function instead of fixing existing one. Percent enconding becomes bad.
It still does not fix potential homoglyph issues, because user can type anything manually.


Support for Zabbix Minor Releases (5.2.x)

Is your feature request related to a problem? Please describe.
I would like to see OPNsense plugins not only support Zabbix LTS but also the minor releases, in this case 5.2.x.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Please provide a package that deploys the latest Zabbix agent.
Describe alternatives you've considered
No other alternatives,
Additional context
Add any other context or screenshots about the feature request here.

https://github.com/opnsense/plugins/pull/2116 Here we discuss solutions, but none really make sense for the release engineering that zabbix does. In the end we may introduce more complexity than we can actually solve due to said upstream process.

+1 on that. Unfortunately zabbix proxy version must be in sync with the server version so single OPNsense instance may be holding back updating of a much bigger network. PfSense manages to provide packages for each supported major release (5.2 and 5.0LTS at the time of writing) and I believe that OPNsense should too, regardless of the implementation.

+1 on that. Unfortunately zabbix proxy version must be in sync with the server version so single OPNsense instance may be holding back updating of a much bigger network. PfSense manages to provide packages for each supported major release (5.2, 5.0LTS, 4.0LTS, 3.0LTS at the time of writing) and I believe that OPNsense should too, regardless of the implementation.

Just made a mistake of going for zabbix 5.2 and can't use the proxy package. I'd love to have it. Seeing that this issue is floating for three months I'll be forced to switch to pfsense for now.

We are certainly not in a rush to make mistakes here. The Zabbix release schedule is not low maintenance for downstream projects so we still have to figure out a viable  way without pushing work our way at the same time. Good luck.

+1 for zabbix-proxy-5.2
Using zabbix in version 5.2 and would like to use the proxy on my opnsense firewalls to monitore some smaller offices.

I wll abuse this issue, since latest LTS of Zabbix is now 5.4 and an upgrade for the plugin would be really nice

Plugin variants are supported now without having to duplicate plugin code. Zabbix proxy is the first to use it, but certainly other zabbix tools can now follow depending on maintainer time.

I plan to add all supported releases of Zabbix Agent in the near future (just need to finish priority tasks first).

Hi @fraenki
as currently already Zabbix 5.4 LTS is released, do you have any expected timeframe?
Appreciate your update!

as currently already Zabbix 5.4 LTS is released

Zabbix 5.4 is not a LTS release:
https://www.zabbix.com/life_cycle_and_release_policy

The plugin was updated with support for plugin variants. It will support both Zabbix 5.0 (LTS) and Zabbix 5.4.
https://github.com/opnsense/plugins/pull/2656
https://github.com/opnsense/tools/pull/251


View Step Count Leaderboard (Team)

working on leaderboard

.deploy noop

.deploy noop

Can you be more descriptive in your PR?  I would like to know what changes I should be seeing in the application when I test your pull request.  I can look at your commit history to see what files you have changed but it makes life a lot harder if I have to piece together what you are working on by myself.


Remove trailing whitespaces

Just removing extra whitespaces

@nishithshah2211,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@nishithshah2211, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

This example is deprecated. Use the language server samples in this repository instead: https://github.com/Microsoft/vscode-extension-samples


[FEA] Support spark.sql.parquet.datetimeRebaseModeInRead=LEGACY

Is your feature request related to a problem? Please describe.
User job failed with the exception:
 org.apache.spark.SparkUpgradeException: You may get a different result due to the upgrading of Spark 3.0:
reading dates before 1582-10-15 or timestamps before 1900-01-01T00:00:00Z from Parquet
files can be ambiguous, as the files may be written by Spark 2.x or legacy versions of
Hive, which uses a legacy hybrid calendar that is different from Spark 3.0+'s Proleptic
Gregorian calendar. See more details in SPARK-31404. You can set the SQL config
'spark.sql.parquet.datetimeRebaseModeInRead' or the datasource option 'datetimeRebaseMode' to 'LEGACY' to rebase the datetime values
w.r.t. the calendar difference during reading. To read the datetime values as it is,
set the SQL config 'spark.sql.parquet.datetimeRebaseModeInRead' or the datasource option 'datetimeRebaseMode' to 'CORRECTED'.

In this case we actually had spark.sql.parquet.datetimeRebaseModeInRead=CORRECTED, but my assumption is since this failed it was overwritten by the setting in the actual parquet file.
It would be nice if the Rapids Plugin could support the LEGACY mode as some CSPs expected that and have it defaults for reads and conversely the write side.

As mentioned in https://github.com/NVIDIA/spark-rapids/issues/9540 , I hope the fix for this issue include below scenarios:
Be able to read an old date/timestamp which was written using spark.sql.parquet.datetimeRebaseModeInWrite=LEGACY.


Windows support

This PR adds bindings for Duktape to Crystal on Windows, this requires changing the use of Make to a more compatible solution: I've gone with Just which is well-made and used heavily within other ecosystems like *nix. This should also work with shards install on Windows (verified locally) but there are a couple of edge-cases.

That's fair, I hadn't considered that. At the same time, this would make the install process longer because of https://github.com/crystal-lang/shards/issues/468. I guess that's not really this shard's problem though, so I'll add the Makefiles back.

Hi @devnote-dev! First off, I just want to say that this is amazing - thanks so much for this work!
Second, I want to apologize for the radio silence. I've been on vacation the past few weeks and am just getting caught up on everything.
Haven't had great chance to review this yet, so please give me a few weeks to get this work reviewed. Rest assured, I'd love to merge any PRs that add support for Windows.
I agree with @z64 around the use of make for *nix platforms - it's the de facto build mechanism for these these platforms.
I'll give this a review over the next few weeks. Thanks again!


The error "delayed_jobs table doesn't exists" when I try to run rails g delayed_job

I followed the instructions showed on README, but, when I run rails g delayed_job as is showed on README the error "delayed_jobs table doesn't exists" happens.
To solve this problem I changed the method self.guess_backend in worker.rb as I show below:
 42     def self.guess_backend
 43       self.backend ||= :active_record if defined?(ActiveRecord) and ActiveRecord::Base.connection.tables.include?('delayed_jobs')
 44     end

Only after I added a "check code" I could continue using delayed job in my project. Someone knows why this error happen, and if this error is a bug?
My environment (using RVM):
ruby 1.9.2-0136
rails 3.0.10

Try requiring 'delayed_job_active_record' instead of delayed_job. That's probably what you're missing, but we need to update the README.
-DM
--
Daniel Morrison
[i] Collective Idea
http://collectiveidea.com
http://ideafoundry.info
25 West 8th Street, Suite 200
Holland, MI 49423
616-990-0155
<EMAIL_ADDRESS>On Dec 11, 2011, at 4:58 PM, Almir 'm3nd3s' wrote:

I follow the instructions showed on README, but, when I run rails g delayed_job as is showed on README the error "delayed_jobs table doesn't exists".
To solve this problem I changed the method self.guess_backend in worker.rb as I show below:
42     def self.guess_backend
43       self.backend ||= :active_record if defined?(ActiveRecord) and ActiveRecord::Base.connection.tables.include?('delayed_jobs')
44     end

Only after I added a "check code" I could continue using delayed job in my project. Someone knows why this error happen, and if this error is a bug?
My environment (using RVM):
ruby 1.9.2-0136
rails 3.0.10

Reply to this email directly or view it on GitHub:
https://github.com/collectiveidea/delayed_job/issues/318

README updated.

Hi @danielmorrison, thanks for help me :)
I tryed using 'delayed_job_active_record' as you sad. The delayed_job (3.0.0.pre4) and delayed_job_active_record (0.2.1) were installed, but when I runned rails g delayed_job to generate the migration and scripts the below error happened:
/Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activerecord-3.0.10/lib/active_record/connection_adapters/abstract/connection_pool.rb:317:in retrieve_connection': ActiveRecord::ConnectionNotEstablished (ActiveRecord::ConnectionNotEstablished) 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activerecord-3.0.10/lib/active_record/connection_adapters/abstract/connection_specification.rb:97:in retrieve_connection'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activerecord-3.0.10/lib/active_record/connection_adapters/abstract/connection_specification.rb:89:in connection' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activerecord-3.0.10/lib/active_record/base.rb:685:in columns'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activerecord-3.0.10/lib/active_record/base.rb:698:in column_names' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activerecord-3.0.10/lib/active_record/relation.rb:370:in block in method_missing'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activerecord-3.0.10/lib/active_record/relation.rb:125:in scoping' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activerecord-3.0.10/lib/active_record/relation.rb:370:in method_missing'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/bundler/gems/searchlogic-92487f9773b0/lib/searchlogic/named_scopes/conditions.rb:92:in condition_details' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/bundler/gems/searchlogic-92487f9773b0/lib/searchlogic/named_scopes/conditions.rb:68:in local_condition?'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/bundler/gems/searchlogic-92487f9773b0/lib/searchlogic/named_scopes/conditions.rb:61:in condition?' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/bundler/gems/searchlogic-92487f9773b0/lib/searchlogic/named_scopes/association_conditions.rb:6:in condition?'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/bundler/gems/searchlogic-92487f9773b0/lib/searchlogic/named_scopes/association_ordering.rb:14:in condition?' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/bundler/gems/searchlogic-92487f9773b0/lib/searchlogic/named_scopes/ordering.rb:11:in condition?'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/bundler/gems/searchlogic-92487f9773b0/lib/searchlogic/named_scopes/ordering.rb:31:in order' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activerecord-3.0.10/lib/active_record/base.rb:447:in order'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/delayed_job_active_record-0.2.1/lib/delayed/backend/active_record.rb:22:in <class:Job>' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/delayed_job_active_record-0.2.1/lib/delayed/backend/active_record.rb:8:in module:ActiveRecord'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/delayed_job_active_record-0.2.1/lib/delayed/backend/active_record.rb:5:in <module:Backend>' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/delayed_job_active_record-0.2.1/lib/delayed/backend/active_record.rb:4:in module:Delayed'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/delayed_job_active_record-0.2.1/lib/delayed/backend/active_record.rb:3:in <top (required)>' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activesupport-3.0.10/lib/active_support/dependencies.rb:239:in require'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activesupport-3.0.10/lib/active_support/dependencies.rb:239:in block in require' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activesupport-3.0.10/lib/active_support/dependencies.rb:225:in block in load_dependency'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activesupport-3.0.10/lib/active_support/dependencies.rb:593:in new_constants_in' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activesupport-3.0.10/lib/active_support/dependencies.rb:225:in load_dependency'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/activesupport-3.0.10/lib/active_support/dependencies.rb:239:in require' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/delayed_job_active_record-0.2.1/lib/delayed_job_active_record.rb:3:in <top (required)>'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/bundler-1.0.18/lib/bundler/runtime.rb:68:in require' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/bundler-1.0.18/lib/bundler/runtime.rb:68:in block (2 levels) in require'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/bundler-1.0.18/lib/bundler/runtime.rb:66:in each' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/bundler-1.0.18/lib/bundler/runtime.rb:66:in block in require'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/bundler-1.0.18/lib/bundler/runtime.rb:55:in each' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/bundler-1.0.18/lib/bundler/runtime.rb:55:in require'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/bundler-1.0.18/lib/bundler.rb:120:in require' 	from /Users/m3nd3s/Projetos/adena/config/application.rb:9:in <top (required)>'
from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/railties-3.0.10/lib/rails/commands.rb:15:in require' 	from /Users/m3nd3s/.rvm/gems/ruby-1.9.2-p136@adena/gems/railties-3.0.10/lib/rails/commands.rb:15:in <top (required)>'
from script/rails:6:in require' 	from script/rails:6:in '

Did you get that error while still having the guess backend method in there?

I'm sorry @bryckbost, I didn't understand what you ask. To test the way as sad by @danielmorrison first I removed the gem delayed_job: gem uninstall delayed_job, so I add the gem instruction in Gemfile and then I runned bundle install. After that I runned rails g delayed_job and I got that error.

I know @bryckbost, and after I added delayed_job_active_record in my Gemfile I didn't change anything in delayed_job gem. But the error, as I sad, happen

I'm not sure if this is the same problem, the generator runs for me, but does not produce a migration:
script/rails generate delayed_job
identical  script/delayed_job
chmod  script/delayed_job

(Ruby 1.9.2-p290, Rails 3.1.3)

@m3nd3s I can't reproduce the issue. I'm going to close this. Feel free to open a new issue if it continues to be a problem.


Is there a plan to enable RabbitMQ Stream API?

Since the RabbitMQ will enable Stream API in next Release 3.9 (master already has this feature), is there a plan to enable Stream API in this repo?

I can't answer for the maintainers of this client of course but we (RabbitMQ team) generally expect streams to have separate dedicated clients as it a completely separate protocol.

Not as part of this client library. It should be a separate RabbitMQ stream protocol client in Go.


Set Content-Type before resolving charset and writing body

Hopefully fixes #346

Thanks!

@hamnis This has been released as part of 2.1.1. It should be available soon.


baro_innov_gate also controls gps gate size for height fusion

Should we really use the baro gate size here, if so we should update the parameter description

Agree, both the variable name and description should be updated. I will create a PR for this.

Fixed in ecl.


Relevant full text search (SQLite FTS5 and/or SQLite extensions)

I am developing an application where I would like to use full text search, and moreover full text search that returns results in order by relevance.
Since I am using a SQLite database, my first thought was to use SQLite's FTS5 built in relevance sort, at least as a baseline, but it appears that the SQLite amalgamation emacsql uses predates FTS5.
My second thought was to use a C extension, but I don't see a way to load an SQLite extension in emacsql. I tried to run  (emacsql db "select load_extension('/path/to/relevance-extension');") but I got a not authorized error.
I was reading through the issue log and it seems like you don't plan to add any new features, so I wonder if you might accept a PR with a solution to one of the above issues. I am not really familiar with the subject matter, but I might be able put something together.
Do you have any advice?

I would really like it if it were updated to use FTS5.  :)  I'll happily forego precompiled binaries as well.

Oh, wait, it already has been?  Slightly confused... does this issue just need to be closed?  Or does the readme need to be updated to mention FTS5, etc?

FTS5 was enabled in 62d3915 three months ago:
https://github.com/skeeto/emacsql/blob/75ac0448a5965c82c616c862cab180c241110fd2/sqlite/Makefile#L6
The issue wasn't closed since extension loading wasn't addressed. Thanks
for pointing out the issue in the README.

Thanks for including the new version of SQLite. I suspect that that will be enough for now, although I hadn't gotten around to evaluating the results since I ran into some other issues (and became personally busy etc).
I will make some time this weekend to get back into what I was doing and confirm if I will do anything right now about loading extensions.


bazecor: 1.3.11 -> 1.4.0

Description of changes
Update bazecor to 1.4.0
https://github.com/Dygmalab/Bazecor/releases/tag/v1.4.0

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

We could also try to patch out the new prompt which offers auto-updates; but given that it is a one time thing (after declining), this doesn't seem too important.

Yeah, frankly I didn't give it much thought. I looked at the appimage and we could patch the autoUpdate with this:
    # Workaround for https://github.com/Dygmalab/Bazecor/issues/370
    postExtract = ''
      substituteInPlace \
        $out/usr/lib/bazecor/resources/app/.webpack/main/index.js \
        --replace-fail \
          'checkUdev=()=>{try{if(c.default.existsSync(f))return c.default.readFileSync(f,"utf-8").trim()===d.trim()}catch(e){u.default.error(e)}return!1}' \
          'checkUdev=()=>{return 1}'

      # Remove the Bazecor autoUpdate message
      substituteInPlace \
        $out/usr/lib/bazecor/resources/app/.webpack/main/index.js \
        --replace-fail \
          'if(void 0===o.get("settings.autoUpdate")){let t;"darwin"!==process.platform?(t={type:"question",buttons:["Decline","Consent"],cancelId:0,defaultId:1,title:"Bazecor auto update tool",message:"Bazecor will update automatically whenever possible",detail:"Do you want to make Bazecor able to auto update automatically?, click consent to authorize automatic downloads of new versions, if you decline this feature, you can enable it again in preferences."},s.dialog.showMessageBox(e,t).then((e=>{1===e.response?o.set("settings.autoUpdate",!0):o.set("settings.autoUpdate",!1)}))):(t={type:"question",buttons:["Consent","Decline"],cancelId:1,defaultId:0,title:"Bazecor auto update tool",message:"Bazecor will update automatically whenever possible",detail:"Do you want to make Bazecor able to auto update automatically?, click consent to authorize automatic downloads of new versions, if you decline this feature, you can enable it again in preferences."},s.dialog.showMessageBox(e,t).then((e=>{0===e.response?o.set("settings.autoUpdate",!0):o.set("settings.autoUpdate",!1)})))}' \
          'o.set("settings.autoUpdate", !1);'
    '';

It replaces the popup's code and disables the autoUpdate settings in the store instead.
I could commit this change, but I'm not sure if this will become more of an headache than anything down the line since patching the appimage is definitely a little jank.
Up to you really, I'm ok with either options.

I have no strong opinion either, let's keep the PR as is then; the line seems a bit long :sweat_smile:


CAMEL-9330 : Main - enableHangupSupport should be default enabled

I can move it to run() or create an internalBeforeStart() and register the shutdown hook there if not siabled

Reworked with an additional method internalBeforeStart so logic is outside run()

Thanks the code has been merged. Do you mind closing


Can't run post request with JSON body for authentication issue even after login success

My script is like below: After login,   self.client.get("/xx/iit/bgc", verify=False) can work well.
But   self.client.request(method="POST", url= "/ata/catas/api", json={"name":"CaNew","description":"DEST"}) always fail for "HTTPError:  511 network authentication is required".
After login, how to pass the authentication for the later request if needed?
from locust import HttpUser, TaskSet, task, between
import urllib3
urllib3.disable_warnings()
import logging
class UserBehaviour(TaskSet):
def on_start(self):
response = self.client.request(method="GET", verify=False, url = "/auth/login/sso?logged_out=true", auth=("user", "password1"))
@task
def some_task(self):
    self.client.get("/xx/iit/bgc", verify=False)
    self.client.request(method="POST", url= "/ta/catas/api", json={"name":"CaNew","description":"DEST"})

class User(HttpUser):
tasks = [UserBehaviour]
wait_time = between(5, 60)

Hi! Please use stackoverflow or slack for questions - thanks!


Allow to specify safeTxGas

Often developers of dapps have a good understanding how much gas is required by their dapps. Therefore for normal transactions it is possible to provide the wallet with a gasLimit that should be used. For the Safe contracts safeTxGas is this gas limit. Therefore we should allow that a sapp can provide this information

I'm running into issues with a safe app where metamask is estimating pretty much bang on the amount of gas required to complete and this is resulting in out-of-gas errors.
Here's an example of a failing and succeeding transaction.
failure (default gas limit): https://rinkeby.etherscan.io/tx/0x074eb6c5761f94892d0022de58a7f7db4633a0a082b4c5130dc64ffb1a3bf101
success (increased gas limit): https://rinkeby.etherscan.io/tx/0xe14bcf80174c845903dfcdc51af500feb40476d61816e8178249b2d060120fdf
I imagine that the root cause is that the 1/64th of gas which is reserved isn't being accounted for so while the overall transaction has enough gas it's just not in the right places.
I'm not really able to tell users increase their gas limits in metamask so being able to set a suggested gas limit which would be passed back to Gnosis Safe ui would be extremely useful in this situation.

This should be related to: https://github.com/gnosis/safe-react/issues/1048 as it seems that the tx submitted doesn't get any gas estimation (aka safeTxGas) at all (it is set to 0 in the transaction)


[BEAM-4494] Migrate recent website changes from beam-site to beam

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

R: @boyuanzz

Ping @boyuanzz; this PR just migrates changes from beam-site repository to beam. This change was automated with a migration script and required no manual changes.


Facebook Authentication Through Firebase

Is your feature request related to a problem? Please describe.
no
Describe the solution you'd like
I would like to add Facebook Authentication Through Firebase Here
Describe alternatives you've considered
Google and email-password authentication already added but not facebook authentication..... I would like to add this.....
Additional context
I already make a Google Authentication pull request here..... I have experience in that kind of authentication.... I think I can do it.....
If I can forward with that.....Please assign me to here....Thanks

Facebook Auth can be implemented. I am assigning this issue to you.
Also have a look on the changes requested in google auth PR, code this similarly.

Facebook Auth can be implemented. I am assigning this issue to you.
Also have a look on the changes requested in google auth PR, code this similarly.

Thanks for assigning me here... First I change in google Auth Pull Request and after merge of that I will start working on facebook auth.... Thanks for review my google auth Pull request.... I surely make changes soon....Thank you


Responsive?

Hi Nitin. The plugin doesn't appear to be responsive. Can it be made responsive using options? I see no mention of that.

I do not want to be impatient, but what is the status about "update on resize"
Thanks

Sorry haven't been able to work on that. Though PR is most welcome.

Hi, just a quick and dirty fix can be found in the small mashup I created if you guys are interested...
http://peterfabian.github.io/justiswipe_flickr_gallery/

@peterfabian Did a little poking around and didn't find your fix. Maybe you could paste the important bits here?

@su-narthur as far as I remember, it's a very simple hack, I just tied the window.onresize event to artificialResponsiveness function in flickr.fetch.js in my repo (which just triggers recalculating of the image sizes and positions).
I think there were some subtle bugs when the re-flow happened around margins, so I attempted to fix that with commit 3c33429, so you might want to play with jquery.justified.images.js a bit, too.


Second search result with same content clears item search select

When the item search select for a shopping list shows items and a second search (e.g. because the user keeps typing) retrieves the same items as a result, the search select is cleared and shows no items at all, although the items are kept in the redux state.

I'm not satisfied with the fix. The result set now flickers for a short time. However, the problem seems to be the component library, so I can't really do anything else


Gherkin-Utils - Java - Address issue #1662 - add a Java pretty print gherkin module

Summary
Ported the TypeScript Gherkin pretty print utility to Java.
Details
Make a java port of the javascript pretty printing library available in gherkin-utils.
Motivation and Context
A huge amount of the Gherkin users out there are java users through cucumber-jvm project. We need a pretty printer for Gherkin.
How Has This Been Tested?
Took most of the unit tests from the typescript project.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).

Checklist:

[x] The change has been ported to Java.
[x] I've added tests for my code.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have updated the CHANGELOG accordingly.

Wow thanks so much for contributing this! I haven't had a chance to look at it yet, but will do my best to do it this week.

As mentioned in https://github.com/cucumber/common/issues/1729 I've literally just finished working on this as an external project 😭 (sorry for not raising an issue to share that I was!)
So I'd be able to help it you'd like with implementation / we can see about pulling my library in here?

As mentioned in #1729 I've literally just finished working on this as an external project 😭 (sorry for not raising an issue to share that I was!)
So I'd be able to help it you'd like with implementation / we can see about pulling my library in here?

That would be great @jamietanna! Just keep in mind that we aim to keep the TypeScript/Java implementations consistent.

Can we please make sure this is testing against everything in gherkin/testadata - looks like escaped_pipes.feature returns a slightly different result that I'd expect (\o vs \o) and there are likely a few other edge cases

The files in gherkin/testadata are (intentionally) not formatted according to the pretty implementation. We can read them all in and format them (the tests in the TypeScript implementation does this), but it doesn't compare the formatted document against anything - we just verify that we can parse the formatted document again.
I think the best way to ensure edge cases are covered is to write dedicated unit tests where we have control over the expected output.

OK do you have a list of the test feature files somewhere you want me to add to the unit tests? I'm happy to do so.
I probably won't finish these updates requested for a few days, doing it in my spare time as my day job is eating up my day on other fires.

Not to worry @nddipiazza!
You can use https://gitlab.com/jamietanna/gherkin-formatter/-/tree/main/src/test/resources/input as the inputs, and https://gitlab.com/jamietanna/gherkin-formatter/-/tree/main/src/test/resources/output as outputs (for passing tests)
Which are following a formatting style I've found useful on my own implementation.
These were largely adapted from https://github.com/cucumber/gherkin-java/tree/master/testdata but I added quite a few additional scenarios as it was required to make sure I had everything really covered.

Excellent. I'm on it.

Feel free to copy code / thinking from the library, too - it's MIT so we can bring it into this project as needed.
There's some additional complexities that don't appear to be in the Typescript version (like comments) so it may be there's a bit of extra work rather than just a plain port :sweat_smile:

Yes, comments are lost in the current implementation, and it would be great to keep them! I think we can do that by popping them off as we traverse the other nodes, comparing line numbers.

Comments seem to be the most difficult piece in the puzzle right now - I'm still trying to get my implementation to handle them right 😥

I've submitted draft PR #1738 to pretty print comments. Feedback welcome!

@aslakhellesoy @jamietanna can you tell when/where this will finally be released? This could become very handy for the cucumber-eclipse plugin as it currently lacks formatting capabilities.

@laeubi in https://www.jvt.me/posts/2021/09/07/gherkin-java-formatter/ I've released a formatter for Gherkin but unfortunately comments are very tricky and I've not quite solved it there yet.
You're welcome to see how it works with your scenarios, but it's not ideal right now 😥

You're welcome to see how it works with your scenarios, but it's not ideal right now disappointed_relieved

Is this released as a maven artifact somewhere or should i simply copy the code into the eclipse-cucumber project for now?

Nice I'll try and have a look later - check https://gitlab.com/jamietanna/gherkin-formatter/-/merge_requests/3 out for some additional edge cases I've run into ☺

FYI, in case you missed it, here's the error in the CI:
touch .rsynced
[[ -z $(git status -s) ]] || ( \
	echo "Working copy is dirty. Please run 'source scripts/functions.sh && rsync_files' and commit modified files." && \
	echo "Found: " && \
	git status -s && \
	exit 1 \
)
Working copy is dirty. Please run 'source scripts/functions.sh && rsync_files' and commit modified files.
Found:
 M gherkin-utils/java/maven-versions-rules.xml
?? gherkin-utils/java/.github/
?? gherkin-utils/java/scripts/
make: *** [Makefile:41: check_synced] Error

Oh gosh, I'm finally getting back to this. I got mixed up with other work I was doing and thought this was merged.
I fixed the comment that was pending and will look into the failing test.

Thanks for picking this up! I’ll let you know when I’ve had a chance to review it.

@jamietanna @aslakhellesoy @aurelien-reeves ready for review.
want me to add any more test cases?

I've continued this work in #1985 - closing this.


when I bower install it, the directory of dist  do not has the css file，hope resolve it

when I bower install it, the  directory of dist do not has the css file，hope resolve it

Are you sure? I just install it, the CSS files existed.


Last time contacted

Does ios have the ability to query the last time contacted? http://developer.android.com/reference/android/provider/ContactsContract.ContactOptionsColumns.html#LAST_TIME_CONTACTED
for my app idea this information would be very nice to have.

I am about 80% sure iOS does not have this options although things may be different in the new Contact Framework (https://developer.apple.com/library/watchos/documentation/Contacts/Reference/Contacts_Framework/index.html)

I'm pretty sure it's not possible on iOS, but on Android I'm pretty sure there are permissions to read their texts and phone call log. That seems out of scope for this repo however, since it's really just a contacts and address book repo.

I think information related to contacts (such as the last time contacted) is within scope for this module but impossible to support on iOS. Perhaps another native module is warranted?

I mean, maybe it's in scope, and I could see an argument for it. It's up to @rt2zz  From my point of view, I wouldn't want to add those permissions to use this repo, since they appear pretty invasive, but if the repo had the functionality I wouldn't be opposed either as long as it was clear I only have to add the permissions to access text messages or phone logs.


GCP - Finish non-IDIR cleanup

Describe the Issue

Still users in GCP that are not managed - reach out to POs to get fixed (have list)
Figure out issues with users that initial try at IDIR integration didn’t work (contractors generally)
Draft comms re: turning off non-IDIR for product teams

Additional Context
https://app.zenhub.com/workspaces/cloud-pathfinder---aws-5e4dbb426c3c6af8dcbf06a7/issues/gh/bcgov/cloud-pathfinder/2865
Acceptance Criteria

[ ] Contact POs for products to remediate non-IDIR users
[ ] Look into specific users that were having issues w/ enforced login (CGI contractors)
[ ] Draft comms for GCP users to enforce IDIR only login (Just Google Cloud, not Google Workspace)

Test with Shelly H.

New ticket working related to unmanaged External Identity (Non Idir)  created : 3030

All the unmanaged Entra Ids converted to managed by organization.


nfs-provisioner vs ganesha fsid=152.152 reserved for internal purposes

ganesha reserved exportID 152 for internal purposes. when generating id for filesystem nfs-provisioner should avoid it.
https://github.com/nfs-ganesha/nfs-ganesha/issues/615
of course having 152 volume claims is an outstanding achievement, but we somehow reached that.

this has to be added to initial setup of ganesha
EXPORT {
    Export_Id=65536;
    Path="/";
    Pseudo="/";
    Access=MDONLY_RO;
    Protocols=4;
    SecType=none,sys,krb5,krb5i,krb5p; # actually just has to list all the SecTypes used by your exports
    Filesystem_Id=65536.65536; # some value that doesn't conflict
    FSAL {
        Name=PSEUDO;
    }
}

Actually nfs-ganesha doesn't reserve export_id 152, but rather, the provisioner is specifying Filesystem_Id as export_id:export_id and nfs-ganesha uses Filesystem_Id 152:152 for the NFS v4 Pseudo Filesystem.
When using FSAL_VFS you should not need to specify Filesystem_Id for exports unless exporting an underlying filesystem that does not have something stable nfs-ganesha can use. nfs-ganesha is capable of using the UUID off the filesystem or the device ID the filesystem is mounted on (which may not be stable, I'm not sure if that's actually a problem for Kubernetes though).

Thanks for reporting the issue!
This repo is no longer being maintained and we are in the process of archiving this repo. Please see https://github.com/kubernetes/org/issues/1563 for more details.
If your issue relates to nfs provisioners, please create a new issue in https://github.com/kubernetes-sigs/nfs-ganesha-server-and-external-provisioner or https://github.com/kubernetes-sigs/nfs-subdir-external-provisioner.
Going to close this issue in order to archive this repo. Apologies for the churn and thanks for your patience! :pray:


Tab name goes missing after some time of usage

Make sure to check the demo app(s) for sample usage
checked the demo app
Make sure to check the existing issues in this repository
there is no issue open /closed
If the demo apps cannot help and there is no issue for your problem, tell us about it
Each Tab has a icon+name. The name goes missing while after some usage of the app / after minimizing the app
Which platform(s) does your issue occur on?

iOS/Android/Both

Please, provide the following version numbers that your issue occurs with:

CLI:3.4.2
Cross-platform modules: 3.1.1
Runtime(s): android-3.4.2 , ios-3.4.1
Plugin(s): 1.3.0

Can you please provide a screen capture?.
PSD: I recommend you to use the latest version of nativescript.

Sorry for the late response. i am attaching a small video of the issue. when i opened my app the names are  missing. When i go to different page and then return, it displays.
IMG_0069.MOV.zip

@Mubashir-Mohamed I saw the video and there is no problem with the plugin, the thing is that you are navigating to a different page. I suggest you see the demo app code where I used the ngSwitch from angular to display components based on the tab selected, so the user will remain in the main view, and you can use the normal navigation router to navigate to a different page like a product detail page, etc.
With the new version of Nativescript, we have the ability to use multiple page-router-outlet that allow us to use that instead of ngSwitch but I didn't have the change to play with it.