Converting CSV to PDF results in a "notSupported" error

I can't use this API endpoint to convert a CSV file to a PDF despite what the doc says. All tries have resulted in an "InputFormatNotSupported" error. It seems like the beta and the 1.0 endpoints are getting mixed up, as the beta doesn't support that sort of conversion but I also get data back if I ask for a JPG file. The 1.0 endpoint definitely doesn't support that. Please update the docs or fix the endpoints to work the way the docs say they will.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 59cfb473-0783-09cb-3c88-96f884d76e06
Version Independent ID: 9992b18b-299d-6863-c06f-8bacb7ff2962
Content: Convert to other formats - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/driveitem-get-content-format.md
Product: sharepoint
Technology: microsoft-graph
GitHub Login: @JeremyKelley
Microsoft Alias: JeremyKelley

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


<EMAIL_ADDRESS>decompress error

Checking hash of Spotube-windows-x86_64-setup.exe ... ok.
Extracting Spotube-windows-x86_64-setup.exe ... ERROR Exit code was 1!
Failed to extract files from C:\ProgramData\Scoop\apps\spotube\3.8.0\Spotube-windows-x86_64-setup.exe.
Log file:
C:\ProgramData\Scoop\apps\spotube\3.8.0\innounp.log

Duplicates #13802.


Feature/physics

How to test:
Make sure that each physics type works properly (powder, liquid, and gas). The powder is white, the liquid is blue, and the gas is red. The yellow and green elements should act as unbreakable solids.
You should also try to create particle outside of the board and make sure that particles aren't created or that segfaults don't occur.
Please report any errors as well as steps to reproduce the error if possible.

Result of my testing. All requested tests pass.
Questions:

Should red (gas) particles end up inside a solid pile or liquid pool?

Otherwise, looks great.

Jake

Result of my testing. All requested tests pass.
Questions:

Should red (gas) particles end up inside a solid pile or liquid pool?

Otherwise, looks great.

Jake

This is normal, I haven't implemented for the particles to swap with each other based on weight, but this will be fixed.


Stronger typing of data

Describe the bug
I want to have stronger type safty than this library offers. Every data creation in v2 has a return type of Any?
In v1 one had toRecord
My code which does this jobb:
class KafkaProducer(
    val kafkaTemplate: KafkaTemplate<String, GenericRecord>
) {
    private val logger = LoggerFactory.getLogger(this::class.java)

    @OptIn(ExperimentalSerializationApi::class)
    fun sendAuditEvent(auditEvent: AuditEvent): Boolean {
        val topic = auditEvent.topic
        val className = auditEvent::class.simpleName
        try {
            if (topic.isEmpty()) {
                throw IllegalArgumentException("Topic for audit event must be set, preferably in the event class")
            }

            val genericData: Any? = Avro.encodeToGenericData(auditEvent)
            kafkaTemplate.send(
                topic,
                UUID.randomUUID().toString(),
                genericData as GenericRecord
            )
            // Returning true if the event was passed along successfully so we can use that in logs
            return true
        } catch (e: Exception) {
            logger.error("Failed to send Audit Kafka event: $className on topic $topic", e)
            // Returning false if the event was not passed along so we can use that in logs
            return false
        }
    }
}

So for the send function which is a powerful method but springframework kafka, I want to be able to not having to cast to a random class.
So just wondering if there are any other method to follow.
To Reproduce
Code to reproduce the behavior:
Use the kode in the explanation
In the explanation

Expected behavior
Simple class passing without unsafe typing
Desktop (please complete the following information):

Kotlin version: 2.0.20
Avro4k version: 1.7.3
Other kotlinx-version: json 1.7.3

Additional context
Add any other context about the problem here.

I'm sorry but I don't see any random class cast, as encodeToGenericData is not able to detect statically if the given content is a data class instance (which is encoded as a GenericRecord), a collection or an array (which is encoded as a GenericArray), a ByteArray (which is encoded as a ByteBuffer), a ByteArray wrapped in a value class having the @AvroFixed annotation (which is encoded as a GenericFixed), or a primitive value.
Your use case is also specific (but common I agree) where your event is a data class which always encode to a GenericRecord. But if you pass one of the other type, you will have something else than a GenericRecord.
Let's take the example of the kotlinx Json library: when you encode any data class using Json.encodeToElement, it always return a JsonElement, where it needs to be cast to JsonObject to allow iteration over fields by example. This is the same as ObjectMapper from jackson, or even the same from the standard java avro library.
Doesn't it sounds right for you ?

As a counter proposition, I would propose you to directly have a KafkaTemplate<String, Any?>, as I suppose you are using confluent schema registry, so everything returned by encodeToGenericData should be fully compatible with the serializer.
It would remove your random cast while still working the exact same as currently 😄

@Chuckame thanks for the diligent feedback and I do agree. Just coming from earlier versions where it was a bit more stricter with the toRecord method. But I think your suggestion might be an valid option yes.
I will try it out :D
Closing for now, coming back if there are any further thoughts


RSS feed blank

RSS feed is blank for the blog itself but works for the author.

https://www.mw-direct.com/blog/rss/
https://www.mw-direct.com/blog/Author/1417/rss/

Umbraco 8.1.3 / Articulate 4.1

Do you have steps to replicate this issue? Might be related to some routing issues I've seen


Consider adopting alloy RPC types

When RPC types become part of alloy, consider adopting those in favour of our own.
Depends On:

https://github.com/alloy-rs/core/pull/51

In our code, we have a preference for strongly typed static types to enforce correctness through the compiler. Alloy's RPC types are not always statically typed. E.g see this type. In addition, it would restrict our flexibility with types, so for now we're choosing not to use Alloy RPC types


jq version requirements should be noted for likely incompatibilities

The latest Ubuntu (23.10) has jq 1.6.
The regular-chatbot exercise documents at least one feature that requires jq 1.7 (the scan/2 function).
It would be helpful to put a blockquote warning in the instructions for cases where current popular operating systems are likely to encounter issues.

Feel free to discuss on the forum.

FYI I am working on some minor changes for this. For example, https://github.com/exercism/jq/blob/904e700aeeba9d6061267465c7971c19a47ef90b/concepts/regular-expressions/introduction.md?plain=1#L122 -- the scan([regex, flags]) form is not implemented for scan, only for test/match/capture.
I'll drop a PR soon and add you both as reviewers.


Add PCL style shared_ptr shortcuts for the implemented registration methods

Its just a minor thing to add and most probably not so many ppl. are using it, but to keep up the PCL implementation style all the specialized registration methods could provide the shared_ptr shortcut ::Ptr.
As an example we can take a look into the public part of pcl::gicp class:
      using Ptr = shared_ptr< GeneralizedIterativeClosestPoint<PointSource, PointTarget> >;
      using ConstPtr = shared_ptr< const GeneralizedIterativeClosestPoint<PointSource, PointTarget> >;

Yeah. There should be Ptr definitions. I'll add Ptr definitions.


Conductor UI - displaying logs does not work

When trying to get logs using the conductor UI, it looks like the URL used to get the logs from the server is wrong - causing the Logs tab to always remain empty.
I have compiled the conductor server and client from the latest git version.
I am creating a very simple task of type SIMPLE and adding logs when updating the task.
If I use Conductor server swagger api:  '/api/tasks/{taskId}/log' - then I can see my logs
When I try to use the conductor UI by selecting the Logs tab I will always get a message stating: 'No logs available'.
When opening the network to see the API used to get the logs I can see the following:
http://localhost:5000/**api/api**/tasks/ (404 not found)
Apparently the code is appending api twice to the url and therefore getting a 404 error

Hi @odedlevy02 , this bug was fixed in v3.4.0, you can upgrade the UI and have a try, thanks.

Thanks but I have taken the latest version with tag v3.4.1 and v2.31.7 (latest main branch) and yet the bug still remains.
I am building a docker image using the docker compose file supplied in folder docker. Building ( docker-compose build ) and running (docker-compose up)

Thanks. It works


HDFS-16985. delete local block file when FileNotFoundException occurred may lead to missing block.

Description of PR
see https://issues.apache.org/jira/browse/HDFS-16985
How was this patch tested?
no unit tests need
For code changes:

If there's only one replica, and AWS EBS fails, doesn't it mean the file is not recoverable anyway?

The EBS not really failed, it may recover soon.
The EBS really failed, we can recover it manually, the data woludn't lost .

Agree that we need to protect data for this case. But the current improvement will leave another issue.
Considering that DataNode notify NameNode without delete block file here, meta at NameNode will be inconsistent at next blockreport round because ReplicaMap is not updated, and it will be involved at next blockreport, right? The result is that NameNode believe this replica is health, but actually it had lost.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 51s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
43m 55s

trunk passed

+1 :green_heart:
compile
1m 34s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
1m 23s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
1m  8s

trunk passed

+1 :green_heart:
mvnsite
1m 34s

trunk passed

+1 :green_heart:
javadoc
1m  9s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 30s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
3m 47s

trunk passed

+1 :green_heart:
shadedclient
26m 41s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 22s

the patch passed

+1 :green_heart:
compile
1m 27s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
1m 27s

the patch passed

+1 :green_heart:
compile
1m 21s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
1m 21s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 56s
/results-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs-project/hadoop-hdfs: The patch generated 1 new + 26 unchanged - 0 fixed = 27 total (was 26)

+1 :green_heart:
mvnsite
1m 20s

the patch passed

+1 :green_heart:
javadoc
0m 53s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 25s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
3m 29s

the patch passed

+1 :green_heart:
shadedclient
25m 36s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
227m 51s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 42s

The patch does not generate ASF License warnings.

347m 38s

Reason
Tests

Failed junit tests
hadoop.hdfs.TestDatanodeReport

hadoop.hdfs.server.namenode.ha.TestObserverNode

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5564

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 62d0c87a431f 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 710fdc5f7eb3b069a0b1322b96a68ff1bdde8186

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/1/testReport/

Max. process+thread count
2310 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

Considering that DataNode notify NameNode without delete block file here, meta at NameNode will be inconsistent at next blockreport round because ReplicaMap is not updated, and it will be involved at next blockreport, right? The result is that NameNode believe this replica is health, but actually it had lost.

@Hexiaoqiao thx for review.
Understand what you mean, removed the replica from ReplicaMap  may work，I will try to fix it.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 47s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 2 new or modified test files.

_ trunk Compile Tests _

-1 :x:
mvninstall
0m 11s
/branch-mvninstall-root.txt
root in trunk failed.

-1 :x:
compile
0m 10s
/branch-compile-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
hadoop-hdfs in trunk failed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.

-1 :x:
compile
0m 10s
/branch-compile-hadoop-hdfs-project_hadoop-hdfs-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
hadoop-hdfs in trunk failed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.

-0 :warning:
checkstyle
0m  7s
/buildtool-branch-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
The patch fails to run checkstyle in hadoop-hdfs

-1 :x:
mvnsite
0m 10s
/branch-mvnsite-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in trunk failed.

-1 :x:
javadoc
0m  9s
/branch-javadoc-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
hadoop-hdfs in trunk failed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.

-1 :x:
javadoc
0m 10s
/branch-javadoc-hadoop-hdfs-project_hadoop-hdfs-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
hadoop-hdfs in trunk failed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.

-1 :x:
spotbugs
0m  9s
/branch-spotbugs-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in trunk failed.

-1 :x:
shadedclient
1m  8s

branch has errors when building and testing our client artifacts.

_ Patch Compile Tests _

-1 :x:
mvninstall
0m 10s
/patch-mvninstall-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch failed.

-1 :x:
compile
0m 10s
/patch-compile-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
hadoop-hdfs in the patch failed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.

-1 :x:
javac
0m 10s
/patch-compile-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
hadoop-hdfs in the patch failed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.

-1 :x:
compile
0m 10s
/patch-compile-hadoop-hdfs-project_hadoop-hdfs-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
hadoop-hdfs in the patch failed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.

-1 :x:
javac
0m 10s
/patch-compile-hadoop-hdfs-project_hadoop-hdfs-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
hadoop-hdfs in the patch failed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m  8s
/buildtool-patch-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
The patch fails to run checkstyle in hadoop-hdfs

-1 :x:
mvnsite
0m 10s
/patch-mvnsite-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch failed.

-1 :x:
javadoc
0m  9s
/patch-javadoc-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
hadoop-hdfs in the patch failed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.

-1 :x:
javadoc
0m 10s
/patch-javadoc-hadoop-hdfs-project_hadoop-hdfs-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
hadoop-hdfs in the patch failed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.

-1 :x:
spotbugs
0m  9s
/patch-spotbugs-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch failed.

-1 :x:
shadedclient
1m 47s

patch has errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
0m  9s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch failed.

+0 :ok:
asflicense
0m 11s

ASF License check generated no output?

7m 17s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5564

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 10e85ecb852c 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / a236be22ed1d29ea905c69a2cece72c493e8e8e8

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/2/testReport/

Max. process+thread count
42 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/2/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@Hexiaoqiao please take a look.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 47s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 2 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
42m 40s

trunk passed

+1 :green_heart:
compile
1m 28s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
1m 22s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
1m  7s

trunk passed

+1 :green_heart:
mvnsite
1m 29s

trunk passed

+1 :green_heart:
javadoc
1m  9s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 32s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
3m 39s

trunk passed

+1 :green_heart:
shadedclient
25m 36s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 19s

the patch passed

+1 :green_heart:
compile
1m 23s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
1m 23s

the patch passed

+1 :green_heart:
compile
1m 15s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
1m 15s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 54s
/results-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs-project/hadoop-hdfs: The patch generated 2 new + 135 unchanged - 0 fixed = 137 total (was 135)

+1 :green_heart:
mvnsite
1m 22s

the patch passed

+1 :green_heart:
javadoc
0m 53s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 26s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
3m 28s

the patch passed

+1 :green_heart:
shadedclient
25m 47s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
228m 45s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 42s

The patch does not generate ASF License warnings.

345m 37s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.datanode.TestDirectoryScanner

hadoop.hdfs.server.namenode.ha.TestObserverNode

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5564

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 2d361c8f8ecf 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / a236be22ed1d29ea905c69a2cece72c493e8e8e8

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/3/testReport/

Max. process+thread count
2006 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/3/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@jojochuang @Hexiaoqiao  can please help take a look ?

@smarthanwang Thanks for your update. it makes sense to me. Would you mind to add unit test to verify if missing block could cut down when the fail volume come back？

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 48s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
45m  7s

trunk passed

+1 :green_heart:
compile
1m 28s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
1m 20s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
1m  7s

trunk passed

+1 :green_heart:
mvnsite
1m 28s

trunk passed

+1 :green_heart:
javadoc
1m  7s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 29s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
3m 37s

trunk passed

+1 :green_heart:
shadedclient
26m  2s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 19s

the patch passed

+1 :green_heart:
compile
1m 21s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

-1 :x:
javac
1m 21s
/results-compile-javac-hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
hadoop-hdfs-project_hadoop-hdfs-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 generated 1 new + 910 unchanged - 0 fixed = 911 total (was 910)

+1 :green_heart:
compile
1m 15s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

-1 :x:
javac
1m 15s
/results-compile-javac-hadoop-hdfs-project_hadoop-hdfs-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
hadoop-hdfs-project_hadoop-hdfs-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09 with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09 generated 1 new + 889 unchanged - 0 fixed = 890 total (was 889)

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 55s
/results-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs-project/hadoop-hdfs: The patch generated 2 new + 170 unchanged - 0 fixed = 172 total (was 170)

+1 :green_heart:
mvnsite
1m 22s

the patch passed

+1 :green_heart:
javadoc
0m 53s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 25s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
3m 27s

the patch passed

+1 :green_heart:
shadedclient
26m  8s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
221m  9s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 42s

The patch does not generate ASF License warnings.

341m 11s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5564

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 92ebb78f487a 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 3f7073b04beacb8af2324455fa63c00b3c4afd9a

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/4/testReport/

Max. process+thread count
2139 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/4/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@Hexiaoqiao  added UT, please take a look~

@smarthanwang Please check result reported by Yetus.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 49s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
42m 52s

trunk passed

+1 :green_heart:
compile
1m 29s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
1m 19s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
1m  6s

trunk passed

+1 :green_heart:
mvnsite
1m 30s

trunk passed

+1 :green_heart:
javadoc
1m  8s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 31s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
3m 35s

trunk passed

+1 :green_heart:
shadedclient
25m 43s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 19s

the patch passed

+1 :green_heart:
compile
1m 23s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
1m 23s

the patch passed

+1 :green_heart:
compile
1m 17s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
1m 17s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 54s
/results-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs-project/hadoop-hdfs: The patch generated 2 new + 170 unchanged - 0 fixed = 172 total (was 170)

+1 :green_heart:
mvnsite
1m 21s

the patch passed

+1 :green_heart:
javadoc
0m 51s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 27s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
3m 30s

the patch passed

+1 :green_heart:
shadedclient
25m 39s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
229m 39s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 42s

The patch does not generate ASF License warnings.

346m 49s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.datanode.TestDirectoryScanner

hadoop.hdfs.TestRollingUpgrade

hadoop.hdfs.server.namenode.ha.TestObserverNode

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/5/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5564

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux e159d3435578 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 1f01e19574164a0c4f1cb16c0b2d8022e4671caa

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/5/testReport/

Max. process+thread count
2224 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/5/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 49s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
42m 25s

trunk passed

+1 :green_heart:
compile
1m 30s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
1m 21s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
1m  6s

trunk passed

+1 :green_heart:
mvnsite
1m 30s

trunk passed

+1 :green_heart:
javadoc
1m  7s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 30s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
3m 37s

trunk passed

+1 :green_heart:
shadedclient
25m 45s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 18s

the patch passed

+1 :green_heart:
compile
1m 24s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
1m 24s

the patch passed

+1 :green_heart:
compile
1m 14s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
1m 14s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 53s

the patch passed

+1 :green_heart:
mvnsite
1m 24s

the patch passed

+1 :green_heart:
javadoc
0m 53s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 27s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
3m 24s

the patch passed

+1 :green_heart:
shadedclient
26m  7s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
224m  9s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 43s

The patch does not generate ASF License warnings.

341m 23s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.datanode.TestDirectoryScanner

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/6/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5564

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 643b4807c65f 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 0bc5fd74e881eb42a0dccf715a903b01ee248422

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/6/testReport/

Max. process+thread count
2016 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/6/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@Hexiaoqiao the failed UT seems not related to this PR，please help check.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 55s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
42m 39s

trunk passed

+1 :green_heart:
compile
1m 29s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
1m 20s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
checkstyle
1m  8s

trunk passed

+1 :green_heart:
mvnsite
1m 30s

trunk passed

+1 :green_heart:
javadoc
1m  9s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 31s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
3m 37s

trunk passed

+1 :green_heart:
shadedclient
25m 49s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 20s

the patch passed

+1 :green_heart:
compile
1m 23s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
1m 23s

the patch passed

+1 :green_heart:
compile
1m 16s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
javac
1m 16s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 55s

the patch passed

+1 :green_heart:
mvnsite
1m 21s

the patch passed

+1 :green_heart:
javadoc
0m 52s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 28s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
spotbugs
3m 31s

the patch passed

+1 :green_heart:
shadedclient
25m 57s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
232m 57s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 42s

The patch does not generate ASF License warnings.

350m 32s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.namenode.ha.TestObserverNode

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/7/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5564

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 4b868cb05963 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / f9462ce5f98b71a20f13a8b257b32368646aaf27

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/7/testReport/

Max. process+thread count
2264 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5564/7/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@Hexiaoqiao fixed the nits，please take a look.

@Hexiaoqiao fixed style issues，please take a look.

LGTM. +1 from my side.

Thanks for detailed review，could you help to do commit ?

@jojochuang I will commit to trunk until wait two work days if no more comments. Thanks.

Committed to trunk. Thanks @smarthanwang for your contribution and thanks @zhangshuyan0 @jojochuang for your reviews.


yarn install is broken

# This file contains the result of Yarn building a package (@expensify/react-native-live-markdown@virtual:7ef6def9554689f801c96d7c46bf0917fbe998740596748f8c9c6e6a3e5a32930c4a319f6524d083f53d2f3c57885ecf9764b1d10daa29c14d4b12000e2eb237#npm:0.1.208)
# Script name: postinstall

command not found: patch-package

Looks like yarn of the consumer calls postinstall on dependencies, causing apps without patch-package installed to break when they install this lib.
To reproduce, run yarn add  this library

Duplicate of https://github.com/Expensify/react-native-live-markdown/issues/573


[Question] Is MVEL evaluation case sensitive ?

I am using MVEL for evaluate java object.
Working:
print org.mvel2.MVEL.eval("?getWrapIn.getAccountNumber", request) ===> "12345"
Not Working:
print org.mvel2.MVEL.eval("?getwrapin.getaccountnumber", request) ===> null
Is there any workaround to make it is behaving same way irrespective of cases.

No way. Mvel is case senstive (as any respectful language should be).


Feature request: Find my mouse toggle or permanently on

Description of the new feature / enhancement
I'd love to have the option to set Find My Mouse permanently on, or at least toggle on/off using double click ctrl. The layout is much less intrusive to the eye and easier to locate than the mouse crosshairs functionality, especially when colors make the crosshairs difficult to see.  Sure you can change the colors of the crosshairs, but you'd have to find a color that doesn't match anything on screen. And when you switch screen image, you'd have to switch crosshairs colors again. The flashlight effect of Find My Mouse always works.
Scenario when this would be used?
It would help a lot in situations where the screen is cluttered with intense graphics. Examples are games that rely on mouse pointer, or when editing videos or photos where patterns make mouse recognition difficult, and you'd need it to be left on for a longer duration.
Supporting information
No response

Thanks for your message. This is a Duplicate of an other issue. Additional comments may be added there. /dup #22561


Fix I/O Bug

Description
Simulation and post_process only read the parallel I/O files up to adv_idx%beg when they should read up to sys_size. This fixes that.
Type of change
Please delete options that are not relevant.

[x ] Bug fix (non-breaking change which fixes an issue)

Scope

[x ] This PR comprises a set of related changes with a common goal

If you cannot check the above box, please split your PR into multiple PRs that each have a common goal.

@sbryngelson Can you rerun the benchmarks, or just merge this without them, given the minimal changes? They timed out due to Phoenix's scratch file system problems.


docs($q): replace "array" with "object" in tests for hash argument

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change?
Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:

Good catch :smiley:
Can you please sign the CLA so we can merge this?

Done!

@gkalpak Is the type "test" or "docs"?

Now it's docs :stuck_out_tongue:
I always have a hard time with these changes, but I think we've been labeling them as docs and since the commit message says docs, let's roll with that :smile:


Append and save uses way too much disk & cpu

Seems to have something to do with block size in the saved format (and attempting to copy blocks instead of copying data within the blocks). This code should run quickly and produce a relatively small file:
import sframe
sf = sframe.SFrame()
sf['foo'] = [0,1,0,0,1,1]
for i in range(17):
    sf = sf.append(sf) # append it to itself to double in size
sf.save('saved_sframe')

Expected:

Saved SFrame should take only a few kilobytes and save almost instantaneously.

Actual:

Saving takes forever (> 3 hours), uses 100% of one core throughout, and produces more than 400 MB of output.

So the size of this SFrame isn't small at all. you're appending an sframe with 6 rows to itself 17 times which gives 6 * 2^17=786432 rows.
That being said, I've seen SFrames of much larger size save quite a bit faster than this. I wonder if it's getting confused when lazily referencing itself too many times or something.

@hershaw: I consider under 1M rows to be "relatively small". 😄 I think there is definitely a bug specifically related to the combination of append and save. I think @ylow investigated and found the root cause but I don't remember what it was. @ylow, has this been fixed in the latest batch of bug fixes?


Added the parametric rayTriangle intersection test to the API as rayTriangleParametric

See https://github.com/AnalyticalGraphicsInc/gltf-pipeline/pull/102.

Update CHANGES.md so users know about this.

Updated! This should be completely covered by the tests for IntersectionTests.rayTriangle. The most significant functional difference between the two is that rayTriangleParametric can return intersections behind the ray origin (negative parametric distance) but that functionality it seems to be covered by the does not intersect behind the ray origin test.

Looks great, thanks @likangning93!


endless service worker updates after first registration without page refresh

Library Affected:
"workbox-background-sync": "^6.5.3",
"workbox-core": "^6.5.3",
"workbox-precaching": "^6.5.3",
"workbox-window": "^6.5.3"
Browser & Platform:
tested on Edge
Issue or Feature Request Description:
After SW registers for the fist time (or user manually unregisters a worker and refreshes the page) -  all following sw version updates will be auto-activated (skipWaiting auto-triggered).
steps to reproduce

sw.js

import { precacheAndRoute, cleanupOutdatedCaches } from "workbox-precaching";

//precacheAndRoute(self.__WB_MANIFEST);
precacheAndRoute([
  { revision: "87f48b43683c695c054d937e20ecf20e", url: "manifest.webmanifest" },
  {
    revision: "1ee7e898708ec49a4f654f623b0579b0",
    url: "assets/index.b806230e.css",
  },
  {
    revision: "74ee3f6b2f7277cd72e41755634f7871",
    url: "assets/index.f4169c83.js",
  },
  { revision: "de1a8c1e5f67be18ed4d254f00217ea6", url: "index.html" },
]);

cleanupOutdatedCaches();

app

import { Workbox, messageSW } from "workbox-window";

let wb;
let registration;
if (process.env.NODE_ENV === "production") {
  if ("serviceWorker" in navigator) {
    isServiceWorkerSupported.value = true;

    wb = new Workbox("/sw.js");

    wb.register({ immediate: false })
      .then((r) => {
        console.log("onRegistered", r);
        setInterval(() => {
          console.log("updating!!");
          r.update();
        }, 10000);
      })
      .catch((e) => {
        serviceWorkerRegistrationError.value = e.message;
        console.error(e);
      });
  }
}

build. Open app first time. Refresh window. This is our starting point.

Change code in app. Build. New SW version shown in chromium debugger:

Verify: no auto-activation triggered.

Change code in app. Build. New SW version shown in chromium debugger:

Verify: no auto-activation triggered.

Unregister service worker in debugger:

Verify: worker status is stopped and deleted

refresh browser window

Verify: version of the website from the last build is shown. Latest sw worker is activated and running.

Change code in app. Build.

Verify [PROBLEM]: Service worker new version got auto-activated (old version of the website is still used, no page refresh was done).

Repeat step 8 as many times  as you want. New SW version will be activated (skipWersion triggered for us). Old site version will keep running until browser window refresh.

why is this a problem?
Because on first page load - if user keeps using this page without refresh - it will keep updating (installing AND activating) to new version forever! I also think that refreshing the page on first activation is bad solution so please do not propose this.
You can verify this problem by opening an incognito window in browser and checking the service worker statuses after new builds (manual de-registration is just a hack if you don't want to deal with incognito)
Another issue is that according to documentation:
https://developer.chrome.com/docs/workbox/reference/workbox-window/

update: function: Checks for updates of the registered service worker.

update function should only invoke new version updates checks, not activate them.
update function works fine as long as you have done the page refresh after first activation. If you didn't - then you deal with the problem from above. :)

Hi there,
Workbox is moving to a new engineering team within Google. As part of this move, we're declaring a partial bug bankruptcy to allow the new team to start fresh. We realize this isn't optimal, but realistically, this is the only way we see it working. For transparency, here're the criteria we applied:

For PRs, we closed everything that doesn't target the current v7 branch.
For Issues, we closed everything that was created before 2024.

Thanks, and we hope for your understanding!
The Workbox team


[IconButton] Custom non-palette color causes TypeError

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
When I augment IconButton with new colors that don't exist in the palette:
declare module "@mui/material/IconButton" {
  interface IconButtonPropsColorOverrides {
    textPrimary: true;
    textSecondary: true;
  }
}

and then try to use these colors:
<IconButton color="textPrimary">
  <CloseIcon />
</IconButton>

My app crashes with:
TypeError: Cannot read properties of undefined (reading 'main')

at the following location:
https://github.com/mui/material-ui/blob/dec32b361714bed9c73ee1360c4b6390769bf9f5/packages/mui-material/src/IconButton/IconButton.js#L78
Expected behavior 🤔
IconButton doesn't throw TypeError when using custom color. Instead it handles it safely, like SvgIcon:
https://github.com/mui/material-ui/blob/dec32b361714bed9c73ee1360c4b6390769bf9f5/packages/mui-material/src/SvgIcon/SvgIcon.js#L54
Steps to reproduce 🕹
No response
Context 🔦
We intend for color="textPrimary" (or color="text.primary", both are fine) to use the text.primary palette color.
Workaround: pass the color to the icon component instead and augment SvgIconPropsColorOverrides typings:
<IconButton>
  <CloseIcon color="textPrimary" />
</IconButton>

Your environment 🌎
No response

The customization capability is not complete. I marked this as an enhancement.
The fix can use optional chaining like in SvgIcon. However, please note that when you customize the color prop, you are opting out of the theme palette so you have to style by:
createTheme({
  components: {
    MuiIconButton: {
      root:  ({ ownerState, theme }) => ({
        ...ownerState.color === 'textPrimary' && {
          // your style
        }
      })
    }
  }
})

Ah that explains it.
I worked around it by passing the color to the icon component instead:
<IconButton>
  <CloseIcon color="textPrimary" />
</IconButton>

Is this issue open to working on?
I'd like to take it. @emlai

@Shubhamchinda I don't think it's being worked on, so feel free to take it. Thank you!

@Shubhamchinda I don't think it's being worked on, so feel free to take it. Thank you!

Great, thanks. I'll take this.

@Shubhamchinda @emlai
Any progress? Would love to implement this PR
@geraldaddey Give me a day, and I'll submit a PR.

Is this issue still open to work on? Would like to work on it

@shachargiladi 4 weeks since the last response, so I think it was abandoned. I'd say go ahead!

Hi is anyone working one this .I'd like to take it. @emlai

Hi is anyone working one this? I'd like to take it.
@emlai

Hi anyone still working on this? I'd like to take it. @emlai @cyberGHostJs @hilalsidhic

Hey, doesn't seems to be an issue for me :thinking:
@emlai

@kushagra010 I started working on it but ran into another error while solving the issue. This is the error
TypeError: Cannot read properties of undefined (reading 'type')
At the following location:
https://github.com/mui/material-ui/blob/0189797d23c6469438c7dc68db214f490697ebce/packages/mui-system/src/colorManipulator.js#L58

is this issue still on or can take it over and work on it?

@harsh5902 you can take it over

I have solved the this problem and created a pull request, please do review it.

@harsh5902 Why was the PR closed?

The PR to fix this issue should:

custom color works with TypeScript
make sure that the custom color does not crash
add a demo to show that the color prop can be augmented

@ZeeshanTamboli if you are available, this could be one to pick up.

@ZeeshanTamboli if you are available, this could be one to pick up.

I am on a bit of a break currently and won't be working next 3-4 weeks much effectively. If somebody else wants to take a look till then, please go ahead.

this is working fine i guess, i just tested it.
at first you need to provide the props in Palette and PaletteOptions interface that lies in the @mui/material/styles/, the color names that you want to add into the project
declare module '@mui/material/styles' {
  interface Palette {
    primaryVariant500: Palette['primary'];
    secondaryVariant500: Palette['primary'];
  }

  interface PaletteOptions {
    primaryVariant500?: PaletteOptions['primary'];
    secondaryVariant500?: PaletteOptions['primary'];
  }
}

and add these lines for IconButton
declare module "@mui/material/IconButton" {
  interface IconButtonPropsColorOverrides {
    primaryVariant500: true;
    secondaryVariant500: true;
  }
}

these enabling the createTheme to be aware of these colors, then you need to provide the implementation in the theme that would be passed in the themeProvider, like so
export const theme = createTheme({
  palette: {
    primaryVariant500: {
      main: '#ff0000'
    },
    secondaryVariant500: {
      main: '#0000ff'
    },
 }
});

because we do want to let the mui know the values of the color.
hope it helps. thanks.

@the-mgi Yes, if the specified color is in your palette then this will work fine.
But we're intending to use color="textPrimary" or color="text.primary" to access the usual text.primary.main that we have in our palette. Currently neither of those work.

@the-mgi Yes, if your palette has the exact same key as the specified color, then this will work fine.
But we're intending to use color="textPrimary" or color="text.primary" to access the usual text.primary.main that we have in our palette. Currently neither of those work.

got it. thanks.

@TCsTheMechanic Workaround for IconButton is to pass the color prop to the icon component instead: https://github.com/mui/material-ui/issues/33054#issuecomment-1148613095

@emlai that also doesn't work

@TCsTheMechanic Did you augment the typings:
declare module "@mui/material/IconButton" {
  interface IconButtonPropsColorOverrides {
    "textColor.light": true;
  }
}

@emlai still nothing, I'm using @mui/icons-material, I don't know if it follows the same module

In our use case the custom color is not in the palette

Then what styles are you applying with the custom color? Why are you calling it with color=textPrimary? What is the use of it?

The same custom color works fine for the icon component.

Yes it works, but if it is not in the palette why should it unnecessarily calculate theme.palette?.color?.main by letting it have nullish check just to prevent crash, but actually not using it?
Am I missing something here?

@ZeeshanTamboli The issue is solid. From my understanding, it is intended that the color prop can be customized from the interface IconButtonPropsColorOverrides but developers get TypeError which is not expected.
This is not about the color value (that's the developers' decision), not our. We just provide a way to extend the color.
This PR will fix the issue: https://github.com/mui/material-ui/pull/34521/

@siriwatknp Hello, it looks like issue still occurs for me. It's working perfectly with Card, SvgIcon and Typography components, but not for Button and IconButton - I have error:
Uncaught TypeError: Cannot read properties of undefined (reading 'type')

I use latest libs versions:
"@mui/icons-material": "^5.11.0",
"@mui/material": "^5.11.4",
"typescript": "^4.9.4",

I tried to reinstall dependencies by removing node_modules and yarn.lock but to no avail.

@siriwatknp Hello, it looks like issue still occurs for me. It's working perfectly with Card, SvgIcon and Typography components, but not for Button and IconButton - I have error:

Uncaught TypeError: Cannot read properties of undefined (reading 'type')

(colorManipulator.js:50)
I use latest libs versions:

"@mui/icons-material": "^5.11.0",

"@mui/material": "^5.11.4",

"typescript": "^4.9.4",

I tried to reinstall dependencies by removing node_modules and yarn.lock but it didn't help.

Pleasr share a Code sandbox that reproduces the issue.

@siriwatknp Facing same issue for icons imported from '@icons-material'. module augmentation is not working for @mui/icons-material. @TCsTheMechanic did you resolve the issue ?

why is the issue closed if the issue is reproducible?

@nikolay-bennie Please provide a reproduction.

Hi @siriwatknp @ZeeshanTamboli
I also got this issue
Uncaught TypeError: Cannot read properties of undefined (reading 'type')

To reproduce:

Create a custom palette color, customColor and assign a random color.
Create a custom IconButton color, customColor, and assign random styles.

Note: The "custom palette color name" should be the same as the "custom IconButton color name".
Here is the code sandbox, try to access /issue route to check the issue.
https://codesandbox.io/p/devbox/nextjs-mui-icon-button-mmml75
I think I did the config correctly, didn't I?


Add support for :PREFERRED_CACHE_HOME:

This PR adds support :PREFERRED_CACHE_HOME: to config.toml path value parsing, so that when the following is set in config.toml: cache_dir = ":PREFERRED_CACHE_HOME:/cointop" the cache files are written to ~/Library/Caches/cointop on macOS.
Additionally I see value in adding /cointop to the default and configured cache directory, unless the last path component is already /cointop, this makes it more clear where the cache files are coming from and allows to clear them more easily as over time many cache files are accumulated.

This is a great improvement @afh! The changes make a lot of sense. I'll add the cointop sub-directory to the cache path
as well which is a good idea.
Merging. Thanks!


Sort array after converting elements to their squares

Enter your question -
Sort the array after converting elements to their squares
Enter link to the question(if question belongs to any online platform) -
https://leetcode.com/problems/squares-of-a-sorted-array/
Tags for the question(eg - Array, Basic, Stack, etc.) -
Array, Sorting

Kindly assign this issue to me @SarthakKeshari

@Anova07,
Kindly add your solution to "leetcode" folder
Deadline - 04/10/2021
This Issue will only be counted towards ranking criteria of this repository.
Gentle advice for futher Issues -
Medium and hard level questions will be given preference for Hacktoberfest tag.

@SarthakKeshari  kindly view my PR if valid under hacktoberfest - PR #221


Make sure we get the ImageMetadata only for image scans

What does this PR do?
Fix a typo introduced in a refactoring PR.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Doesn't show any extension

Issue Type: Feature Request
My visual studio code doen't show any extension
VS Code version: Code 1.48.2 (a0479759d6e9ea56afa657e454193f72aef85bd0, 2020-08-25T10:13:11.295Z)
OS version: Windows_NT x64 10.0.19041

/~needsMoreInfo


Update links

Avalanche education (learn) goes to "about" but should go here: https://nwac.us/avalanche-education/
Membership: https://nwac.us/membership/
Volunteer: https://support.nwac.us/volunteer/
I think support can be combined with memberships? And be called "become a member" or "donate" or "support" ... I can't decide what the right one is right at the moment.
latest news: can we link to a section on the website? Or just the URL? news lives on the www.nwac.us homepage

not really up to us to make a final decision 😄 i just stubbed in a few things that seemed reasonable
https://github.com/NWACus/avy/blob/f35c53a6de01a4f4e5937f2ca8d7e0cd5773d0c6/data/settingsMenuItems.ts#L24-L37

Yes, sorry, not asking for you to make a decision, just whether the royal we had made one :)

Ah, not that I know of! I think @charlotteguard was in charge of tracking the things we need from SNFAC:

square logo (unless the one we hacked in works)
the app links above
description shown in the center picker
...anything else I'm forgetting?

It might be easier to get further along and ask for all the things at once, as opposed to having a bunch of separate email threads to track.

closing this issue as the original work is completed - we'll track the SNFAC stuff separately


SPIKE: Research is it possible to force HTTP 1.1 for EventStream in a browser

So currently when using EventStream it crashes as because of the http2 dependency. Please check if:

it is possible to support http2 in a browser (I don't see any polyfills)
it is possible to force http 1.1 + ssl in a browser

Because when calling curl -vso /dev/null --http2 'https://events.mainnet.casperlabs.io/events/main' I see that both http 1.1 and http2 is supported:
~ → curl -vso /dev/null --http2 'https://events.mainnet.casperlabs.io/events/main'
*   Trying <IP_ADDRESS>:443...
* Connected to events.mainnet.casperlabs.io (<IP_ADDRESS>) port 443 (#0)
* ALPN: offers h2
* ALPN: offers http/1.1
*  CAfile: /etc/ssl/cert.pem
*  CApath: none

@gyroflaw If this can not be fixed, we need to revert the change and get a more sustained fix.

ALPN: offers http/1.1

* ALPN, server accepted to use h2
* Server certificate:
*  subject: CN=*.mainnet.casperlabs.io
*  start date: Feb 10 00:00:00 2023 GMT
*  expire date: Jan 30 23:59:59 2024 GMT
*  subjectAltName: host "events.mainnet.casperlabs.io" matched cert's "*.mainnet.casperlabs.io"
*  issuer: C=US; O=Amazon; CN=Amazon RSA 2048 M01
*  SSL certificate verify ok.
* Using HTTP2, server supports multiplexing
* Connection state changed (HTTP/2 confirmed)
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* TLSv1.2 (OUT), TLS header, Supplemental data (23):

It seems that this load balancer using HTTP2

*Using HTTP2, server supports multiplexing
*Connection state changed (HTTP/2 confirmed)

And consider this module: https://github.com/davedoesdev/browser-http2-duplex can be used in our SDK

Currently the eventstream ALB supports http1.1 protocol.


Add local API authorization

Codecov Report

Merging #2975 (ae63b16) into master (f3a12a9) will decrease coverage by 0.33%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #2975      +/-   ##
============================================
- Coverage     59.59%   59.26%   -0.34%
+ Complexity     4458     4438      -20
============================================
  Files           695      695
  Lines         20074    20104      +30
  Branches       1807     1810       +3
============================================
- Hits          11964    11914      -50
- Misses         7097     7176      +79
- Partials       1013     1014       +1

Impacted Files
Coverage Δ

...enyu/admin/controller/DashboardUserController.java
92.00% <ø> (ø)

...u/admin/service/impl/DashboardUserServiceImpl.java
83.33% <ø> (ø)

.../org/apache/shenyu/common/config/ShenyuConfig.java
0.00% <ø> (ø)

.../java/org/apache/shenyu/common/utils/ShaUtils.java
80.00% <ø> (ø)

...pache/shenyu/web/filter/LocalDispatcherFilter.java
0.00% <0.00%> (ø)

...e/shenyu/plugin/base/fallback/FallbackHandler.java
61.53% <0.00%> (-38.47%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
68.75% <0.00%> (-31.25%)
:arrow_down:

...henyu/admin/service/impl/UpstreamCheckService.java
42.85% <0.00%> (-12.70%)
:arrow_down:

...shenyu/register/client/http/utils/OkHttpTools.java
44.44% <0.00%> (-10.73%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

... and 41 more

Continue to review full report at Codecov.

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

please check ci error

please check ci error

Thanks, I have fixed it.


Update to fix 2 Laravel 5.4 issues

This pull request updated the read me and service provider.

👍

@sngrl any chance this could get merged?


Using map_err over inspect_err

This patch fixes clippy::manual-inspect error. See [1] for details.
[1] https://rust-lang.github.io/rust-clippy/master/index.html#/manual_inspect

Maybe log the es instead of remove them? Seems useful.
LGTM either way

Thanks for the review!
Those errors are lib libcubeb's errors like InvalidFormat or InvalidParameter, not providing helpful information for debugging, so I didn't log them. I'll leave them as they are for now.


fix: json body prettify not working with comments

Added Microsoft's node-jsonc-parser library that is able to format json with comments. Seems to handle bigints properly too.
It may be good to consider replacing existing decomment library with jsonc-parser if its only use case is to decomment json.
Fixes usebruno#1830
Before. Failing to prettify JSON body with comments:

After. JSON body properly formatted:

Description

Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

Merged!
Thank you @asonkeri !


Feature Request: Function List pane

Add the ability to show sticky pane on the right side that shows a list/overview of things like functions and declarations for the current file, allowing to user to click on them to quickly goto the appropriate code

I think this is related to https://github.com/zyedidia/micro/pull/977 because having a support for a language server will allow to parse AST and get all the members of a file.

I've implemented similar functionality in micro-jump plugin, which is now in the official plugin channel.
You can press F4 to get a list of all functions for the current file, select a function and jump to it.
$ sudo apt-get -y install micro fzf ctags
$ micro -plugin install jump

It's based on ctags, so it supports 40+ languages, such as Markdown, Go, C, C++, Java and Python.
https://github.com/terokarvinen/micro-jump


make failed

after the yaml installation thing it says make failed

Your system is missing the libyaml library or development header.


I want to add the ability to push to remote branches

I want to add the ability to push to remote branches
Now there's only push, no remote branch shown.
Reference pictures

None of this is true.
My idea is to push it to a remote branch.

I'm not sure what that editor is, but it definitely doesn't seem like VS Code.


No metabase rebuild

We use to rebuild it to change logging levels, but that is not necessary anymore since we only follow init logs during start. We also switch to v44.1

@thomas-gerber Something broke - https://github.com/faros-ai/faros-community-edition/actions/runs/3492542071

it is just flaky. Removing the validation of all those sources in Airbyte will solve this, I think.


Upstream certificate validation broken in 0.15

In fixing #1165 I'm pretty sure that Upstream Certificate validation was broken. The symptom is SDS will not return a the CA certificate referenced in by CDS's upstream TLS context.

investigating a bit more I think this was broken when we introduced SDS in 0.13.

Phew. Nothing is broken. When we switch to SDS we didn't convert Cluster Upstream Validation to use SDS. I've raised #1405 to track this as tech debt.


Golang errors in file: [demos/demo#3/pull-model/producer.go]:

  > [line 37 : column 6] - ClusterReport redeclared in this block
  > [line 270 : column 14] - cannot convert report (variable of type ClusterReport) to []byte
  > [line 297 : column 6] - main redeclared in this block

Already fixed in #233


Remove URL parameters before evaluating the file name.

Closes #120

Thanks for your contribution @mcrowson !
Would it be possible to add a test for this fix?
Also, the string manipulation is not needed as parsed_uri.uri_path already contains the path without parameters. For an unknown reason it is overwritten here, but that should probaby be removed without consequences.

Sure, I'll write a test and go with that approach. Commits should be incoming later today.
Sent from my iPhone

On Apr 27, 2017, at 2:42 AM, Lev Konstantinovskiy<EMAIL_ADDRESS>wrote:
Thanks for your contribution @mcrowson !
Would it be possible to add a test for this fix?
Also, the string manipulation is not needed as parsed_uri.uri_path already contains the path without parameters. For an unknown reason it is overwritten here, but that should probaby be removed without consequences.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

It looks like it is being overwritten, because the full URI needs to be passed to the HttpOpenRead function. My presigned file for example wouldn't work without the parameters. Because the function exists outside of the smart_open class we don't have access to self either to see the parsed_uri object.
I think just parsing the URL again within that function is the cleanest and most consistent option. Either that or the string function, or finding a way to pass the parsed URL from here to the function.

No, the issue this is related to is because there are URL parameters which means the .gz is in the middle of the string.
Sent from my iPhone

On May 2, 2017, at 8:51 PM, Lev Konstantinovskiy<EMAIL_ADDRESS>wrote:
@tmylk commented on this pull request.
In smart_open/smart_open_lib.py:

@@ -754,11 +754,11 @@ def HttpOpenRead(parsed_uri, mode='r', **kwargs):

 response = HttpReadStream(url, **kwargs)

fname = url.split('/')[-1]

fname = urlsplit(url, allow_fragments=False).path.split('/')[-1]
May I ask about the reason for the .split('/')[-1]?
Would just a if split_url.path.endswith('.gz') check be enough?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Not sure why travis timed out at Py3.6, works locally on 3.6. Closing and reopening for it to try the build again.

Thanks a lot @mcrowson ! Clean job.


replace git.NewCommand() with git.NewCommandContext()

as title

I propose to use only NewCommand because otherwise every method will end in ...Context sometimes. As the context parameter should be the new normal, we could drop the Context suffix.

Also need to get rid of the commands like CloneWithContext etc.

@zeripath can not find CloneWithContext and I think it's for a next pull :)

Looks like I might have fixed that in the previous pr.

I propose to use only NewCommand because otherwise every method will end in ...Context sometimes. As the context parameter should be the new normal, we could drop the Context suffix.

I think so. NewCommand is a better name than NewCommandContext.

@lunny already changed :D

Title should be changed.

@KN4CK3R suggestions for a better one are welcome

I would just have switched both methods but the current one is ok too. 👍

replace git.NewCommandContext() with git.NewCommand()

Codecov Report

Merging #18552 (3943c22) into main (8ae5e6d) will decrease coverage by 0.00%.
The diff coverage is 60.18%.

@@            Coverage Diff             @@
##             main   #18552      +/-   ##
==========================================
- Coverage   46.30%   46.29%   -0.01%
==========================================
  Files         846      846
  Lines      121241   121239       -2
==========================================
- Hits        56140    56129      -11
- Misses      58289    58298       +9
  Partials     6812     6812

Impacted Files
Coverage Δ

cmd/hook.go
7.11% <0.00%> (ø)

modules/doctor/mergebase.go
10.25% <0.00%> (ø)

modules/doctor/misc.go
21.55% <0.00%> (ø)

modules/git/pipeline/lfs_nogogit.go
0.00% <0.00%> (ø)

modules/git/pipeline/namerev.go
0.00% <0.00%> (ø)

modules/indexer/code/bleve.go
63.36% <0.00%> (ø)

modules/indexer/code/elastic_search.go
1.12% <0.00%> (ø)

routers/private/hook_verification.go
0.00% <0.00%> (ø)

routers/web/repo/pull.go
30.63% <0.00%> (ø)

services/asymkey/sign.go
38.46% <0.00%> (ø)

... and 64 more

Continue to review full report at Codecov.

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


Returns cidrs that are already in use.

It worked the first time. But now when using in a new stack it returns a cidr that is either identical or within the range from the results provided to the first stack that used it.
`
The CIDR '<IP_ADDRESS>/24' conflicts with another subnet
`

Hmm. I can't replicate this. I wonder if you pulled the repo when there was a bug. I've added tags now and a bunch more unit tests so I'd suggest trying again with 2.0.0.
I'll close this for now but please reopen if this is still an issue :)


configurable wrapping token TTL

Currently, the TTL of the wrapping token of the secret-id is fixed to 60 seconds.
Are there any strong opinions against making it configurable?
We'd like to leave unwrapping to our applications, but unfortunately, 60 seconds is too short in some cases.

This is now configurable using the vault.wrappingTTL configuration option: https://github.com/Boostport/kubernetes-vault#vault-required which is released in v0.6.1.

Wow that was fast, thanks a bunch!


Bridging the gap in network overhead measurement in the profiler

Signed-off-by: Raj<EMAIL_ADDRESS>Description
For profiler queries on multi-node clusters, we get a timing breakdown on a shard-by-shard basis regarding time spent in various sub parts of the query. However, this neglects the time spent on each shard query in transit over the network in the case of multi node clusters.
This PR resolves this by adding additional fields for each shard to get the time spent in transit over the network (in milliseconds).
Issues Resolved
Resolves #1258
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

start gradle check

In Log 698,
2 test failures of "mixedClusterTest"
> Task :qa:mixed-cluster:v1.0.1#mixedClusterTest

REPRODUCE WITH: ./gradlew ':qa:mixed-cluster:v1.0.1#mixedClusterTest' --tests "org.opensearch.backwards.IndexingIT.testSeqNoCheckpoints" -Dtests.seed=D0951238EDA1DCAE -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1" -Dtests.locale=zh-TW -Dtests.timezone=America/Paramaribo -Druntime.java=17

org.opensearch.backwards.IndexingIT > testSeqNoCheckpoints FAILED
    org.opensearch.client.ResponseException: method [GET], host [http://[::1]:39387], URI [test/_count?preference=_only_nodes%3Av1.0.1-1], status line [HTTP/1.1 500 Internal Server Error]
    {"error":{"root_cause":[{"type":"e_o_f_exception","reason":null}],"type":"search_phase_execution_exception","reason":"all shards failed","phase":"query","grouped":true,"failed_shards":[{"shard":0,"index":"test","node":"DenRzxzWTE26zLeVzUtvdg","reason":{"type":"e_o_f_exception","reason":null}}],"caused_by":{"type":"e_o_f_exception","reason":null,"caused_by":{"type":"e_o_f_exception","reason":null}}},"status":500}
        at __randomizedtesting.SeedInfo.seed([D0951238EDA1DCAE:E1F9F8F3273E59F2]:0)
        at org.opensearch.client.RestClient.convertResponse(RestClient.java:345)
        at org.opensearch.client.RestClient.performRequest(RestClient.java:315)
        at org.opensearch.client.RestClient.performRequest(RestClient.java:289)
        at org.opensearch.backwards.IndexingIT.assertCount(IndexingIT.java:299)
        at org.opensearch.backwards.IndexingIT.testSeqNoCheckpoints(IndexingIT.java:241)

> Task :qa:mixed-cluster:v1.0.1#mixedClusterTest

REPRODUCE WITH: ./gradlew ':qa:mixed-cluster:v1.0.1#mixedClusterTest' --tests "org.opensearch.backwards.IndexingIT.testIndexVersionPropagation" -Dtests.seed=D0951238EDA1DCAE -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1" -Dtests.locale=zh-TW -Dtests.timezone=America/Paramaribo -Druntime.java=17

org.opensearch.backwards.IndexingIT > testIndexVersionPropagation FAILED
    org.opensearch.client.ResponseException: method [GET], host [http://[::1]:40131], URI [indexversionprop/_count?preference=_only_nodes%3Av1.0.1-3], status line [HTTP/1.1 500 Internal Server Error]
    {"error":{"root_cause":[{"type":"illegal_state_exception","reason":"unexpected byte [0xe8]"}],"type":"search_phase_execution_exception","reason":"all shards failed","phase":"query","grouped":true,"failed_shards":[{"shard":0,"index":"indexversionprop","node":"OduwdcTZTTmEfFMsDcNhLA","reason":{"type":"illegal_state_exception","reason":"unexpected byte [0xe8]"}}],"caused_by":{"type":"illegal_state_exception","reason":"unexpected byte [0xe8]","caused_by":{"type":"illegal_state_exception","reason":"unexpected byte [0xe8]"}}},"status":500}
        at __randomizedtesting.SeedInfo.seed([D0951238EDA1DCAE:5106461848571AF1]:0)
        at org.opensearch.client.RestClient.convertResponse(RestClient.java:345)
        at org.opensearch.client.RestClient.performRequest(RestClient.java:315)
        at org.opensearch.client.RestClient.performRequest(RestClient.java:289)
        at org.opensearch.backwards.IndexingIT.assertCount(IndexingIT.java:299)
        at org.opensearch.backwards.IndexingIT.testIndexVersionPropagation(IndexingIT.java:126)

And several others for "internalClusterTest", such as
> Task :server:internalClusterTest

REPRODUCE WITH: ./gradlew ':server:internalClusterTest' --tests "org.opensearch.search.aggregations.bucket.HistogramIT.testEmptyWithExtendedBounds" -Dtests.seed=D0951238EDA1DCAE -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1" -Dtests.locale=sk-SK -Dtests.timezone=America/Yakutat -Druntime.java=17

org.opensearch.search.aggregations.bucket.HistogramIT > testEmptyWithExtendedBounds FAILED
    java.lang.AssertionError: Unexpected ShardFailures: [shard [[08af6keRS0qJHspUk1TBag][idx][5]], reason [RemoteTransportException[[node_s1][<IP_ADDRESS>:45239][indices:data/read/search[phase/query]]]; nested: NullPointerException[Cannot invoke "org.opensearch.search.internal.ShardSearchRequest.setOutboundNetworkTime(long)" because the return value of "org.opensearch.search.query.QuerySearchResult.getShardSearchRequest()" is null]; ], cause [java.lang.NullPointerException: Cannot invoke "org.opensearch.search.internal.ShardSearchRequest.setOutboundNetworkTime(long)" because the return value of "org.opensearch.search.query.QuerySearchResult.getShardSearchRequest()" is null

@dblock - Could you elaborate on which areas I could add tests for? I can't do any specific value testing for time spent on network and there are checks in place for negative values and different versions (for mixed-cluster).

I see several behavior changes:

SearchExecutionStatsCollector sets outbound network time to 0 vs. not under certain conditions
SearchQueryThenFetchAsyncAction sets inbound networking time under certain conditions
ShardSearchRequest, ProfileShardResult and SearchProfileShardResults serialize inbound/outbound networking time
...

Generally, I'd expect to see a test that shows that profiler queries no longer neglect the time spent on each shard query in transit over the network in the case of multi node clusters - a test that would fail without this change.


Add terminationMessagePolicy: FallbackToLogsOnError to fix observability-termination-policy tnf test

build-depends: https://github.com/dci-labs/dallas-pipelines/pull/965

SUCCESS https://www.distributed-ci.io/jobs/6ecd3c0e-dda4-4063-94b1-4d4379247381/jobStates
from Add terminationMessagePolicy: FallbackToLogsOnError to fix observability-termination-policy tnf test #30

There are still two containers failing in this test, I'll recheck and rerun the job:
 "NonCompliantObjectsOut": [
    {
      "ObjectType": "Container",
      "ObjectFieldsKeys": [
        "Reason For Non Compliance",
        "Namespace",
        "Pod Name",
        "Container Name"
      ],
      "ObjectFieldsValues": [
        "TerminationMessagePolicy is not FallbackToLogsOnError",
        "example-cnf",
        "testpmd-app-77bb9c5c85-pkj8p",
        "testpmd"
      ]
    },
    {
      "ObjectType": "Container",
      "ObjectFieldsKeys": [
        "Reason For Non Compliance",
        "Namespace",
        "Pod Name",
        "Container Name"
      ],
      "ObjectFieldsValues": [
        "TerminationMessagePolicy is not FallbackToLogsOnError",
        "example-cnf",
        "testpmd-app-77bb9c5c85-t6j5r",
        "testpmd"
      ]
    }
  ]

In the current job that is being executed, which is about to finish, observability-termination-policy test is passing, you can see it here: https://www.distributed-ci.io/jobs/359b6279-5ef8-40ae-9240-136581e9b9b9/tests
The change is ready for review.


Integrate "courtney" with pkg-go.

This tool adds support for marking code blocks as uncoverable and excludes some blocks that are pretty hard/irrelevant to cover.
https://github.com/dave/courtney

At first glance, it looks like you have to run your test suite through courtney for this to work at all, not by running go test.  This isn't necessarily a problem, but I would prefer if it could be done as a post-processing pass on the coverage reports, if possible.

/cc @danilvpetrov

Nevermind, it doesn't support modules (see https://github.com/dave/courtney/issues/16).
This would be an interesting project to try to recreate, but I don't want to get sidetracked right now.


Appending values to existing entries via extra to structure_logs(append=True) entries

Is your feature request related to a problem? Please describe.
After having set an attribute globally:
e.g.
LOGGER.structure_logs(append=True,
                        context={"device_name": device_name})

I would like to log additional items to the context attribute e.g.
LOGGER.info("hello world",
            extra={"context": {"device_location": "UK"}})

Describe the solution you'd like
LOGGER.info("hello world")  would include a JSON payload with both device_location and device_name in the above example whereas currently it overwrites the context attribute with the new entry,
Describe alternatives you've considered
I could do:
LOGGER.info("hello world",
            extra={"context": {"device_name": device_name,
                                        "device_location": "UK"}})

But if the device_name is not globally available throughout the code base, it is not ideal.

Closing as we haven't had an update ever since. Please please do reopen if you have answers or thoughts to those questions.
Thank you!!


Changes for compatibility with Ansible > 2.4, fixes for #211

What does this PR do?
Makes snaps-openstack to work with ansible > 2.4
Do you have any concerns with this PR?
Testing with other versions different than 2.7.2 of ansible
How can the reviewer verify this PR?
Install ansible 2.7 and run
Any background context you want to provide?
Screenshots or logs (if appropriate)
Questions:

Have you connected this PR to the issue it resolves?
Does the documentation need an update?
Does this add new Python dependencies?
Have you added unit or functional tests for this PR?
Does this patch update any configuration files?

@platini2 I agree with @spisarski, especially due to the vulnerability issue with ansible version <2.4.5 (see https://github.com/cablelabs/snaps-openstack/issues/206)

Ok, so if we agree that, I can remove all the logic to be backward compatible to support ansible < 2.4

reverted previous commits and included changes in a single commit to make it compatible with ansible > 2.4 and require ansible over 2.4.5

There are some other pep8 failures but not on the files that I changed in this commit.

@spisarski @bo-quan It looks like the pep8 issues were introduced in a different PR.  Can we approve this PR, since it didn't add the pep8 issues?


Algolia integration

Tracker story link: https://www.pivotaltracker.com/story/show/144892879
Main goals:
1 - better matching
2 - autosuggest
3 - show which sections/parts of the page matched
4 - be able to customize how we score title vs body
QA Instructions:
I've deployed this branch to: https://dox-orientation-qa.herokuapp.com/
As far as testing the goals: go the 'Articles' tab and for each goal, do the following:
1 - search something in the search bar, you should get more relevant results verses our previous PSQL text search, e.g. type in 'onboarding'
2 - start typing something, you should start to see suggestions of articles that correspond to what you're typing
3 - as you type something, that part should be bolded in the suggestions that pop up
4 - this isn't really testable, but titles are set to be more important than body, so if you type a word (e.g. 'Gemfury') that appears in the title of article 1 and then also shows up in the body of article 2, article 1 should be higher up in suggestions/results

Ready for deploy


Use SourceKind::diff for formatter

Summary
This PR refactors the formatter diff code to reuse the SourceKind::diff logic. This has the benefit that the Notebook diff now includes the cell numbers which was not present before.
Test Plan
Update the snapshots and verified the cell numbers.

thanks!


Platformatic client generator should be runtime aware

The generator for the client cannot currently fetch the schema from within a non-exposed service of Platformatic Runtime

Related to this, the client generator shouldn't add anything to .env.sample or .env if it's generating a client within a Runtime project.

I think these are the things that need to be covered for the Platformatic Client generator to be "Runtime aware":

It should be able to accept a service ID instead of a URL, if it's being run anywhere inside of a Runtime application project (this would probably be simpler if the client CLI accepted named arguments instead of positional arguments).
It should be able to access the service's OpenAPI schema even if the service isn't exposed by the Runtime app (as in this issue's description).
When updating a service's configuration, it should set a serviceId in the clients object for the client, not a url.
If it's being run inside of a service directory, it should not add anything to the service's .env.sample or .env files (as mentioned in my earlier comment).

Did I miss anything?


added Agora dependancy to Gradle

Added missing dependency, README states dependency is there but it wasn't.

(This sample program has added this address and do not need to add again. Adding the link address is the most important step if you want to integrate the Agora Video SDK in your own application.)

Sorry for the inconvenience, I have already update the README file. Please keep tell us things you found not friendly to developers or wrong information. Thanks


website updates

:warning: Only to be merged at around 9am Pacific Time.

The changes to the docs shall be reverted in about a week.
The hero section can maybe be reset to what it was before in a month...?

→ https://deploy-preview-1289--fluxcd.netlify.app/
→ https://deploy-preview-1289--fluxcd.netlify.app//docs (and subpages)
→ https://deploy-preview-1289--fluxcd.netlify.app/blog

On a larger monitor, the text in the announcement bar moves but is off centre:

As opposed to:

Should be fixed now.


Menu: target should lead to top page

When clicking on documentation left menu, the reader should land on the top of the page, not at the bottom.

Resolved with new documentation portal.


Adding camel-jackson dependency creates issues with inner classes

When adding the camel-jackson dependency, comments with imports, actual imports, etc inner classes are no longer found.
version: Camel K Client and Operator 1.0.0-RC2
Example: https://github.com/jeremyrdavis/quarkus-cafe-demo/commits/camel-k-addition
- commit works: d95b5ea360ca1dabe333b2ce568fc87b18e8594d
- commit does not work: 4ee524f4ccf2c9dce3005b80e7fa44b4d1546528
Process on OpenShift 4.4:

Create project
Install camel k operator
kamel install --olm=false --skip-cluster-setup --skip-operator-setup --maven-repository  https://jitpack.io@id=jitpack@snapshots
kamel run --name=rest-with-undertow --dependency=camel-rest --dependency=camel-undertow --dependency=mvn:com.github.jeremyrdavis:quarkus-cafe-demo:1.2-SNAPSHOT RestWithUndertow.java

This should not be an issue any-more in camel-k 1.0.0 ?
The issues was caused by a bug in the jOOR library which is used to compile the Java integration at runtime which is included in jOOR's 0.9.13 [1]. The camel-k-runtime version used by camel-k RC2 is
1.1.0 [2] which dos not includes the fix [3]
[1] https://github.com/jOOQ/jOOR/issues/99
[2] https://github.com/apache/camel-k/blob/1.0.0-RC2/script/Makefile#L19
[3] https://github.com/apache/camel-k-runtime/blob/camel-k-runtime-parent-1.1.0/pom.xml#L42
@mmistretta can you please confirm ?

i used this commit: https://github.com/jeremyrdavis/quarkus-cafe-demo/commit/5e8c7c47f7082a68e2c30dd7adcea0cad4bce56e
and still get Caused by: java.lang.ClassNotFoundException: RestWithUndertow.GrubHubOrder

@mmistretta I've finally been able to find the root cause, here the related issues:

https://issues.apache.org/jira/browse/CAMEL-15199
https://github.com/apache/camel-k-runtime/issues/364

I think we can close this now that we have all the pieces released


wrong registration of dHCP T2

Hello
thanks for providing this code, I am very exited to test it. (nice work !)
I just tried on a dHCP subject, but the registration.py you provide failed to find the appropriate scaling ...  why did you choose an adult size mni template ?
I am surprise since you train and evaluated on dHCP dataset.
did you register the sub-CC01234BN14_ses-155230_T2w.nii.gz to the template in your code base only with the register.py script ?
Many thanks for your help
Romain

Hi Romain,
Thanks very much for your interest in our work.
The register.py script is based on ANTsPy and may fail on some subjects. My suggestion is to change the aff_metric parameter in ants.registration and re-run the script for the failed subjects. Alternatively, you can run the affine registration between the binary brain masks instead of the MRI intensity. Hope these can help.
We chose the MNI-152 template so that CoTAN can be easily generalized and trained on adult datasets as well. Sorry for the inconvenience caused.
Qiang


Chore: Updates for 6.4

With this pull request the template's dependencies are updated for 6.4

Self merging this


Actions for the MySQL charm

It is unclear to me what happens to an old superuser after creating a new one

At the end this action creates the new user and runs: GRANT ALL PRIVILEGES ON *.* TO 'username'@'%'


panic running dep ensure

What version of dep are you using (dep version)?
-> % dep version
dep:
 version     : v0.4.1
 build date  : 2018-01-27
 git hash    : 37d9ea0
 go version  : go1.9.3
 go compiler : gc
 platform    : darwin/amd64

What dep command did you run?
dep ensure
-> % dep ensure
panic: version queue is empty, should not happen

goroutine 1 [running]:
github.com/golang/dep/gps.(*solver).findValidVersion(0xc4201f0240, 0xc4202eb9a0, 0xc42043b9b0, 0x1, 0x1, 0x0, 0xc42043b9b0)
	/private/tmp/dep-20180127-38430-r7nlwc/src/github.com/golang/dep/gps/solver.go:927 +0x53f
github.com/golang/dep/gps.(*solver).createVersionQueue(0xc4201f0240, 0xc420166f60, 0x14, 0x0, 0x0, 0xc42043b9b0, 0x1, 0x1, 0x0, 0x0, ...)
	/private/tmp/dep-20180127-38430-r7nlwc/src/github.com/golang/dep/gps/solver.go:914 +0x955
github.com/golang/dep/gps.(*solver).solve(0xc4201f0240, 0x17e7e40, 0xc4200160d0, 0x0, 0xc420268de0, 0xc420268e40)
	/private/tmp/dep-20180127-38430-r7nlwc/src/github.com/golang/dep/gps/solver.go:517 +0x5b0
github.com/golang/dep/gps.(*solver).Solve(0xc4201f0240, 0x17e7e40, 0xc4200160d0, 0x1843a90, 0xc420010054, 0x20, 0xc4203cccf0)
	/private/tmp/dep-20180127-38430-r7nlwc/src/github.com/golang/dep/gps/solver.go:457 +0xe4
main.(*ensureCommand).runDefault(0xc420157f46, 0xc42005d080, 0xc42000a0a0, 0x0, 0x0, 0xc42005d0e0, 0x17eb320, 0xc42005d440, 0xc420010044, 0x30, ...)
	/private/tmp/dep-20180127-38430-r7nlwc/src/github.com/golang/dep/cmd/dep/ensure.go:302 +0x168
main.(*ensureCommand).Run(0xc420157f46, 0xc42005d080, 0xc42000a0a0, 0x0, 0x0, 0x0, 0x0)
	/private/tmp/dep-20180127-38430-r7nlwc/src/github.com/golang/dep/cmd/dep/ensure.go:220 +0x8f4
main.(*Config).Run(0xc42005cf60, 0xc42005cf60)
	/private/tmp/dep-20180127-38430-r7nlwc/src/github.com/golang/dep/cmd/dep/main.go:210 +0x11cb
main.main()
	/private/tmp/dep-20180127-38430-r7nlwc/src/github.com/golang/dep/cmd/dep/main.go:52 +0x1fc

What did you expect to see?
Dep updates the lock file
What did you see instead?
Panic with no action to take. It would be convenient if the import causing the issue was included in the message. I've commented out all of the Gopkg.toml imports and it is still throwing these panics.
[prune]
  go-tests = true
  unused-packages = true
  non-go = true

  [[prune.project]]
    name = "github.com/wellington/go-libsass"
    non-go = false
    unused-packages = false #libsass-build must exist but can not be imported directly

Here's the gopkg.toml with all constraints commented out.

It seems something has broken with resolvinggopkg.in/fsnotify/fsnotify.v1. Here's what it looks like in dep sources
[~/pkg/dep/sources/https---gopkg.in-fsnotify.v1] [master]
-> % git remote show origin
* remote origin
  Fetch URL: git@github.com:go-fsnotify/fsnotify
  Push  URL: git@github.com:go-fsnotify/fsnotify

Here's go get
-> % go get -u -v go get gopkg.in/fsnotify/fsnotify.v1
package get: unrecognized import path "get" (import path does not begin with hostname)
Fetching https://gopkg.in/fsnotify/fsnotify.v1?go-get=1
Parsing meta tags from https://gopkg.in/fsnotify/fsnotify.v1?go-get=1 (status code 200)
get "gopkg.in/fsnotify/fsnotify.v1": found meta tag get.metaImport{Prefix:"gopkg.in/fsnotify/fsnotify.v1", VCS:"git", RepoRoot:"https://gopkg.in/fsnotify/fsnotify.v1"} at https://gopkg.in/fsnotify/fsnotify.v1?go-get=1
gopkg.in/fsnotify/fsnotify.v1 (download)

-> % cd ~/src/gopkg.in/fsnotify/fsnotify.v1
drew@dumass [21:44:01] [~/src/gopkg.in/fsnotify/fsnotify.v1] [master]
-> % git remote show origin
* remote origin
  Fetch URL: https://gopkg.in/fsnotify/fsnotify.v1
  Push  URL: https://gopkg.in/fsnotify/fsnotify.v1

At some point, fsnotify moved from github.com/go-fsnotify/fsnotify to github.com/fsnotify/fsnotify. Not sure why dep is still seeing the old location. Wiping the cache or manually fixing up the repo in dep/sources doens't resolve the issue.

Appears, there is no clever resolution happening. We just need to implement the user resolution
gopkg.in/pkg.v3      → github.com/go-pkg/pkg (branch/tag v3, v3.N, or v3.N.M)
gopkg.in/user/pkg.v3 → github.com/user/pkg   (branch/tag v3, v3.N, or v3.N.M)

Dep was officially deprecated earlier this year, and the proposal to archive this repository was accepted. As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!


How to hide a sheet?

Hi,
I need help, I want to hide a sheet that I have protected, this sheet contains information that I should not be visible to others users.  I don't find a way to implement this.
Thanks.

Try worksheet.state = 'hidden'


execute() only pulls maximum of 10,000 rows when no parameter is supplied

cnt_by_destIP.info()
Table rows: 251613

Column             Type    Non-null #

sensor_id          int32   251613
dest_ip            string  251613
unique_flow_count  int64   251613
source_bytes       int64   251613
dest_bytes         int64   251613

cnt_by_destIP.execute().shape
(10000, 5)

cnt_by_destIP.execute(500000).shape
(251613, 5)

The documentation on this function is incorrect, specifying that supplying no parameter will export the entire table, where clearly this is not the case. Please correct the code so that the condition specified in the documentation is how the execute() method operates.

hi Matt,
Here's what the documentation says currently:
        Parameters
        ----------
        limit : integer or None, default 'default'
          Pass an integer to effect a specific row limit. limit=None means "no
          limit". The default is whatever is in ibis.options.

So to get the whole table, you can pass expr.execute(limit=None). Patches welcome on this and any other documentation issues -- let me know if you found wrong information someplace else